make jshint happy

Author: Narretz

src/ng/directive/select.js

@@ -504,7 +504,6 @@ var selectDirective = function() {
         // Read value now needs to check each option to see if it is selected
         selectCtrl.readValue = function readMultipleValue() {
           var array = [];
-          var options = element.find('option')
           forEach(element.find('option'), function(option) {
             if (option.selected && !option.disabled) {
               var val = option.value;
@@ -572,6 +571,7 @@ var optionDirective = ['$interpolate', function($interpolate) {
       var interpolateValueFn, interpolateTextFn;
 
       if (isDefined(attr.ngValue)) {
+        // jshint noempty: false
         // Will be handled by registerOption
       } else if (isDefined(attr.value)) {
         // If the value attribute is defined, check if it contains an interpolation

test/ng/directive/selectSpec.js

@@ -1488,7 +1488,7 @@ describe('select', function() {
           optionElements = element.find('option');
           expect(optionElements.length).toEqual(3);
           expect(scope.obj.value).toBe(null);
-          expect(element.val()).toBe(prop === 'ngValue' ? '? object:4 ?' : '? string:A ?')
+          expect(element.val()).toBe(prop === 'ngValue' ? '? object:4 ?' : '? string:A ?');
       });
 
 
@@ -1538,7 +1538,7 @@ describe('select', function() {
           optionElements = element.find('option');
           expect(optionElements.length).toEqual(4);
           expect(scope.obj.value).toBe(null);
-          expect(element.val()).toBe('? string:A ?')
+          expect(element.val()).toBe('? string:A ?');
       });
 
 
@@ -1631,7 +1631,7 @@ describe('select', function() {
           B.disabled = true;
           scope.$digest();
 
-          var optionElements = element.find('option');
+          optionElements = element.find('option');
           expect(optionElements.length).toEqual(4);
           expect(optionElements[0].value).toEqual(unknownValue(undefined));
 
@@ -2034,7 +2034,7 @@ describe('select', function() {
             D.disabled = true;
             scope.$digest();
 
-            var optionElements = element.find('option');
+            optionElements = element.find('option');
             expect(optionElements.length).toEqual(4);
             expect(element[0].value).toBe('');