Use brackets for default values instead of parens

* Closes #303
* Documentation: SassDoc/sassdoc.github.io#78

Author: valeriangalliat

src/annotation/annotations/parameter.js

@@ -1,4 +1,4 @@
-let typeRegEx = /^\s*(?:\{(.*)\})?\s*(?:\$?([^\s]+))?\s*(?:\((.*)\))?\s*(?:-?\s*([\s\S]*))?/;
+let typeRegEx = /^\s*(?:\{(.*)\})?\s*(?:\$?([^\s]+))?\s*(?:\[([^\]]*)\])?\s*(?:-?\s*([\s\S]*))?/;
 
 export default function parameter() {
   return {

src/annotation/annotations/property.js

@@ -1,4 +1,4 @@
-let reqRegEx = /\s*(?:\{(.*)\})?\s*(?:(\$?[^\s]+))?\s*(?:\(([^\)]*)\))?\s*(?:-?\s*([\s\S]*))\s*$/;
+let reqRegEx = /\s*(?:\{(.*)\})?\s*(?:(\$?[^\s]+))?\s*(?:\[([^\]]*)\])?\s*(?:-?\s*([\s\S]*))\s*$/;
 
 export default function property() {
   return {

src/annotation/annotations/require.js

@@ -1,7 +1,7 @@
 let utils = require('../../utils');
 let uniq = require('lodash').uniq;
 
-let reqRegEx = /^\s*(?:\{(.*)\})?\s*(?:(\$?[^\s]+))?\s*(?:\((.*)\))?\s*(?:-?\s*([^<$]*))?\s*(?:<?\s*(.*)\s*>)?$/;
+let reqRegEx = /^\s*(?:\{(.*)\})?\s*(?:(\$?[^\s]+))?\s*(?:\[([^\]]*)\])?\s*(?:-?\s*([^<$]*))?\s*(?:<?\s*(.*)\s*>)?$/;
 
 export default function (config) {
   return {

test/annotations/parameter.test.js

@@ -8,8 +8,8 @@ describe('#parameter', function () {
   var param = (new (require('../../dist/annotation').default)()).list.parameter;
 
   it('should return an object', function () {
-    assert.deepEqual(param.parse('{type} $hyphenated-name (default) - description'), { type: 'type', name: 'hyphenated-name', default: 'default', description: 'description' });
-    assert.deepEqual(param.parse('{type} $name (()) - description'), { type: 'type', name: 'name', default: '()', description: 'description' });
+    assert.deepEqual(param.parse('{type} $hyphenated-name [default] - description'), { type: 'type', name: 'hyphenated-name', default: 'default', description: 'description' });
+    assert.deepEqual(param.parse('{type} $name [default] - description [with brackets]'), { type: 'type', name: 'name', default: 'default', description: 'description [with brackets]' });
     assert.deepEqual(param.parse('{List} $list - list to check'), { type: 'List', name: 'list', description: 'list to check' });
   });
 
@@ -19,11 +19,11 @@ describe('#parameter', function () {
   });
 
   it('should work for multiline description', function () {
-    assert.deepEqual(param.parse('{type} $hyphenated-name (default) - description\nmore\nthan\none\nline'), { type: 'type', name: 'hyphenated-name', default: 'default', description: 'description\nmore\nthan\none\nline' });
+    assert.deepEqual(param.parse('{type} $hyphenated-name [default] - description\nmore\nthan\none\nline'), { type: 'type', name: 'hyphenated-name', default: 'default', description: 'description\nmore\nthan\none\nline' });
   });
 
   it('should work without the $', function () {
-    assert.deepEqual(param.parse('{type} hyphenated-name (default) - description\nmore\nthan\none\nline'), { type: 'type', name: 'hyphenated-name', default: 'default', description: 'description\nmore\nthan\none\nline' });
+    assert.deepEqual(param.parse('{type} hyphenated-name [default] - description\nmore\nthan\none\nline'), { type: 'type', name: 'hyphenated-name', default: 'default', description: 'description\nmore\nthan\none\nline' });
   });
 
 });

test/annotations/property.test.js

@@ -24,23 +24,23 @@ describe('#property', function () {
       description: 'description'
     });
 
-    assert.deepEqual(prop.parse('{Function} base.default (default) - description'), {
+    assert.deepEqual(prop.parse('{Function} base.default [default] - description'), {
       type: 'Function',
       name: 'base.default',
       default: 'default',
       description: 'description'
     });
 
-    assert.deepEqual(prop.parse('{Function} base.default (default) - description (with parens)'), {
+    assert.deepEqual(prop.parse('{Function} base.default [default] - description [with brackets]'), {
       type: 'Function',
       name: 'base.default',
       default: 'default',
-      description: 'description (with parens)'
+      description: 'description [with brackets]'
     });
   });
 
   it('should work for multiline description', function(){
-    assert.deepEqual(prop.parse('{Function} base.default (default) - description\nmore\nthan\none\nline'), {
+    assert.deepEqual(prop.parse('{Function} base.default [default] - description\nmore\nthan\none\nline'), {
       type: 'Function',
       name: 'base.default',
       default: 'default',

test/data/test.scss

@@ -67,7 +67,7 @@
  * - `@throw`
  *
  * @param {*} $arg - Whatever
- * @param {List} $extra-arguments (()) - Extra arguments
+ * @param {List} $extra-arguments [()] - Extra arguments
  *
  * @return {*} Anything
  *
@@ -85,7 +85,7 @@
  *
  * @content Content is parsed and whatever.
  *
- * @param {Number} $number (42) - Number
+ * @param {Number} $number [42] - Number
  * @param {Arglist} $extra-arguments - Extra arguments
  *
  * @output Nothing
@@ -100,8 +100,8 @@
  * - `@prop`
  * - `@type`
  *
- * @prop {String} base.first-key ("default") - Description
- * @prop {String} base.second-key (42) - Description
+ * @prop {String} base.first-key ["default"] - Description
+ * @prop {String} base.second-key [42] - Description
  *
  * @type {*}
  */