Making REGEX_OPTIONAL greedy

[timdp]

Allows for arrays in default values for optional arguments. E.g., @param {Array} [arr=[]]. The current version would leave out the closing bracket.

[okuryu]

@timdp Thanks for your contributions! I'm not able to reproduce your problem. Could you provide example code which occurs the problem?

[timdp]

Does @param {Array} [arr=[]] produce the right output for you? With the existing regexp, we would get arr=[ without the matching brace.

[okuryu]

Sorry! I just confirmed your problem and it was resolved by your patch. So here is my example code (index.js).

/**
 * @module MyModule
 * @class MyClass
 */

/**
 * @method func1
 * @param {Array} [option1=[]] The option 1
 */
function func1(option1) {
}

The results by latest YUIDoc.

and data.json as follows.

$ cat out/data.json | jq .classitems[0].params[0].optdefault
"["

The results by your patch.

and data.json as follows.

$ cat out/data.json | jq .classitems[0].params[0].optdefault
"[]"

Am I correct?

[timdp]

Yep, that's what I had in mind.

[yahoocla]

CLA is valid!

[okuryu]

@timdp I just merged, thanks for your contribution!

[timdp]

You're welcome!