Ok, I have pushed the 'JSHint - gjslint' commits.
The framework classes are now clean against this gjslint command:
gjslint --strict --disable 0100 -r ./
I've used the --disable command because the new interface handling
relies on a variable on the prototype that is initialised with a 'non
primitive' value.
I also ran the framework against a very loose set of JSHint tests
using this 'header':
/*jshint
curly: false,
eqeqeq: false,
eqnull: true,
globalstrict: true,
indent: 2,
maxlen: 120,
onevar: false,
strict: true,
white: false */
/*global goog, org */
'use strict';
I had to use the loose set because the coding styles differ too much
between files to allow stricter checks. I would suggest that we comply
with both the above mentioned 'gjslint' arguments and with the
following JSHint 'header':
/*jshint
globalstrict: true,
indent: 2,
maxlen: 120,
strict: true,
white: false */
/*global goog, org */
'use strict';
I suggest we use maxlen 120 to avoid a warning for long lines caused
by long 'goog.require' statements. 'gjslint' takes care of the 80 char
enforcement.
EdB
On Thu, Nov 7, 2013 at 8:57 AM, Alex Harui <[email protected]> wrote:
>
>
> On 11/6/13 11:41 PM, "Erik de Bruin" <[email protected]> wrote:
>
>>Alex,
>>
>>Do you also see the following warning when compiling DataBindingTest to
>>JS:
>>
>>Data binding will not be able to detect assignments to 'strings'.
>>
>>dataProvider="{MyModel(applicationModel).strings}" />
> Yes.
>>
>>Just checking to make sure I have the same results as you... Also,
>>when I run the app (debug and release), I see the string 'undefined'
>>displayed several times on screen. Is that what you see as well?
> No. Where do you see that? Did you get my changes? Databinding was
> broken because the is/as code didn't handle "x as String". I put a new
> FlexJSOverlay.zip on my people.a.o folder. Maybe you need that.
>
> -Alex
>
--
Ix Multimedia Software
Jan Luykenstraat 27
3521 VB Utrecht
T. 06-51952295
I. www.ixsoftware.nl
