[patch] [cla] dojo/_base argument comments fix

[Stephen Chung]

Fixes some types in argument comments.

Puts in comments where missing.

[bill]

Note that if the data types should only be listed once, for example for dojo.byId() we already have:

/*=====
dojo.byId = function(id, doc){
	// id: String|DOMNode
	//	 	A string to match an HTML id attribute or a reference to a DOM Node
	// doc: Document?
	//		Document to work in. Defaults to the current value of
	//		dojo.doc.  Can be used to retrieve
	//		node references from other documents.
=====*/

So we don't need:

dojo.byId = function(/*String|DOMNode*/id, /*Document?*/doc){ 

Or is that for the benefit of closure compiler?

[Stephen Chung]

Bill, you're correct. Sorry I forgot to remove those few patches that have corresponding comment blocks. :-)

Note: only the closure compiler needs JsDoc? comments. However, it can also be generated from the comment block in a post-processing pass in addition to in-line argument type comments.

[Eugene Lazutkin]

Replying to schungx:

Bill, you're correct. Sorry I forgot to remove those few patches that have corresponding comment blocks. :-)

Are you going to update your patch? Or is it good already?

[Stephen Chung]

Patch updated. Unfortunately I neglected to click the "replace" checkbox, so now the new patch is in "dojo._base.2.patch"

"dojo.place" and "dojo.create" have argument types in the comment block below the function declaration but not in-line with the arguments. Not sure whether the comment block counts as duplicated type comments, so I just left the two lines in the patch. It appears to be more the standard style these days with in-line type comments and no type in the block comment below the function declaration.

[Eugene Lazutkin]

Fixed in [24740].