[[FIX]] Relax restriction on module option

While the `module` option is intended to be set before executable code,
this restriction should only be enforced when it is first enabled (and
not with the application of every in-line directive).

src/jshint.js

@@ -198,13 +198,6 @@ var JSHINT = (function() {
     }
 
     if (state.option.module) {
-      /**
-       * TODO: Extend this restriction to *all* "environmental" options.
-       */
-      if (!hasParsedCode(state.funct)) {
-        error("E055", state.tokens.next, "module");
-      }
-
       /**
        * TODO: Extend this restriction to *all* ES6-specific options.
        */
@@ -630,6 +623,15 @@ var JSHINT = (function() {
           return;
         }
 
+        if (key === "module") {
+          /**
+           * TODO: Extend this restriction to *all* "environmental" options.
+           */
+          if (!hasParsedCode(state.funct)) {
+            error("E055", state.tokens.next, "module");
+          }
+        }
+
         var match = /^([+-])(W\d{3})$/g.exec(key);
         if (match) {
           // ignore for -W..., unignore for +W...

tests/unit/options.js

@@ -2930,5 +2930,13 @@ exports.module.declarationRestrictions = function( test ) {
       "/* jshint module: true */"
     ], { esnext: true });
 
+  TestRun(test, "Not re-applied with every directive (gh-2560)")
+    .test([
+      "/* jshint module:true */",
+      "function bar() {",
+      "  /* jshint validthis:true */",
+      "}"
+    ], { esnext: true });
+
   test.done();
 };