Ignore jshint directives

[bernharduw]

directives for jshint should be ignored:

  /*jshint latedef:false */
  var define = require('amdefine')(module);

results in

{
    "tags": [],
    "description": {
      "full": "<p>shint latedef:false</p>",
      "summary": "<p>shint latedef:false</p>",
      "body": ""
    },
    "ignore": false,
    "code": "var define = require('amdefine')(module);\n}\n\n\ndefine(function (require) {\n\t'use strict';\n\tvar _ = require('underscore');\n\t\n\tvar type = function(typename,
 required, options) {\n\t\t\tvar t = {type: typename};\n\t\t\tif (required) {\n\t\t\t\tt.required = true;\n\t\t\t}\n\t\t\treturn _.extend(t, options);\n\t};\n\t\n\tvar combine = 
function(",
    "ctx": {
      "type": "declaration",
      "name": "define",
      "value": "require('amdefine')(module)",
      "string": "define"
    }
  }

Those directives should really be treated as regular code. JSHint does not allow a space between the comment start and jshint or global:

ok: /*jshint latedef:false */, /*global console:true */
not recognized by jshint: /* jshint onevar:false */

[phun-ky]

👍

[tj]

haha sounds reasonable :D

[shannonmoeller]

👍

[jmendiara]

👍

[yangit]

[tregusti]

This is still an issue with /*global thingie*/. It becomes lobal thingie. Personally I think dox should ignore anything not starting with /** and ending with */.

[Twipped]

Are you certain that you are using version 0.5.1?

The test .parseComments() code with a multi-line comment on a single line test specifically checks for that style of comment, and the test is passing correctly. I even changed it specifically to /*global thingie*/ to confirm, and intentionally broke the test to make sure that it wasn't falsely passing.

[tregusti]

I'm sorry. I was not. I tried with grunt-doxx, that depended on doxx, that depended on dox ~0.4.4.