Mailing-List: contact callback-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: callback-dev@incubator.apache.org
Received-SPF: pass (athena.apache.org: domain of gtanner@gmail.com designates
 209.85.219.47 as permitted sender)
MIME-Version: 1.0
Sender: gtanner@gmail.com
In-Reply-To: <CC936316.F4CD%fil@adobe.com>
References: 
 <CALxOMKDxBohOa8D_4Mu9B2-O3D=HCTcWLvnTjU+YuOt6nB5Aeg@mail.gmail.com>
 <CC936316.F4CD%fil@adobe.com>
From: Gord Tanner <gord@tinyhippos.com>
Date: Fri, 5 Oct 2012 10:30:33 -0400
Message-ID: 
 <CAESi+49XHgxhV9ips0eZhLi_zJpT6QxQBoL43nWAGzVqwfwxLw@mail.gmail.com>
Subject: Re: Lint errors - bootstrap, tizen ContactUtils, win8 MediaFile
To: callback-dev@incubator.apache.org
Content-Type: text/plain; charset=ISO-8859-1

+11

I pulled in the simple lint changes since it is a huge win.


On Thu, Oct 4, 2012 at 7:01 PM, Filip Maj <fil@adobe.com> wrote:
> Just here to +1 fixing lint errors. Would love to get a patch in to
> eliminate the win8 + tizen JS from warning me about lint errors when I
> build the js :o
>
> On 10/4/12 1:04 PM, "Mark Dineen" <mark@dineen.biz> wrote:
>
>>Hi everyone,
>>
>>I'm working on linting cordova-js.  I submitted a pull with the (mostly)
>>obvious ones, but wanted to run the three in this commit by the list
>>before
>>issuing a pull.
>>
>>https://github.com/mdineen/incubator-cordova-js/commit/198ebe97ba3c38d9bb7
>>6f94d421812c912c46829
>>
>>bootstrap - jshint complained that the function CordovaNavigator is
>>declared in a block.  I moved it to the top and grepped for other
>>instances, I found no conflicts.
>>
>>Tizen ContactUtils - to me, it appeared that initializeAllFieldsMapping
>>needed to run once and only once, so I changed it to anonymous and removed
>>the code that clears the local
>>
>>Win8 MediaFile - jshint complained that contentType was undeclared.
>> Grepping around it seemed quite likely that the proper reference for this
>>should be storageFile.contentType.
>>
>>Please comment on these if you have any concern with my approach.
>>
>>I'm new.  My ICLA has been on file since Sep 10, 2012.  I'm a contributor
>>to Ripple, working independently on Cordova.
>>
>>Thanks,
>>
>>Mark.
>


-- 
Gord Tanner
Senior Developer / Code Poet
tinyHippos Inc.
@tinyhippos