@lint ignoreUndefined(...) broken?

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

@lint ignoreUndefined(...) broken?

Florin Jurcovici
It seems to no longer work.

It bloats my ./generate lint output. I have several tests in which I define test-only classes in setUp(). I had   @lint ignoreUndefined(...) for the methods using these classes everywhere, to keep the lint output clean. After I upgraded from 2.0 to 2.1, these annotations stopped working. Is this intentional or should I file a bug report?

--
A swindler is arrested for  selling an immortality drug to people. At the station, policemen are surprised to find it's not the guy's first offense, charges for the same offense being filed against him as early as 1651 ...


------------------------------------------------------------------------------
Symantec Endpoint Protection 12 positioned as A LEADER in The Forrester  
Wave(TM): Endpoint Security, Q1 2013 and "remains a good choice" in the  
endpoint security space. For insight on selecting the right partner to
tackle endpoint security challenges, access the full report.
http://p.sf.net/sfu/symantec-dev2dev
_______________________________________________
qooxdoo-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
Reply | Threaded
Open this post in threaded view
|

Re: @lint ignoreUndefined(...) broken?

Thomas Herchenroeder

On 03/07/2013 07:55 PM, Florin Jurcovici wrote:
> It seems to no longer work.
>
> It bloats my ./generate lint output. I have several tests in which I
> define test-only classes in setUp(). I had   @lint
> ignoreUndefined(...) for the methods using these classes everywhere,
> to keep the lint output clean. After I upgraded from 2.0 to 2.1, these
> annotations stopped working. Is this intentional or should I file a
> bug report?


Could you post a snippet of code where you used the @ignore tag and
where you used the corresponding class?! It might depend on the lexical
context.

T.


------------------------------------------------------------------------------
Symantec Endpoint Protection 12 positioned as A LEADER in The Forrester  
Wave(TM): Endpoint Security, Q1 2013 and "remains a good choice" in the  
endpoint security space. For insight on selecting the right partner to
tackle endpoint security challenges, access the full report.
http://p.sf.net/sfu/symantec-dev2dev
_______________________________________________
qooxdoo-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
Reply | Threaded
Open this post in threaded view
|

Re: @lint ignoreUndefined(...) broken?

Florin Jurcovici
Sorry for answering that late, I just got other stuff on my hands and forgot about this, I just stumbled upon the problem again, and Google was smart and led me to my own post.

Here's a test that reproduces the problem - it runs through, but lint complains about demoapp.test.DemoTestClass being undefined when generate lint:

qx.Class.define("demoapp.test.DemoTest",
{
        extend : qx.dev.unit.TestCase,

        members :
        {
                __defineLintWarningTestClass: function()
                {
                        qx.Class.define("demoapp.test.DemoTestClass", {
                                extend: qx.core.Object,
                                members:
                                {
                                        saySomething: function()
                                        {
                                                var message = this.classname + " instance is doing well.";
                                                this.debug(message);
                                                console.log(message);
                                        }
                                }
                        });
                },
               
                /*
                 * @lint ingoreUndefined(demoapp.test.DemoTestClass)
                 * @lint ingoreUndefined(DemoTestClass)
                 */
                testLocallyDefinedClassFiresLintWarning: function()
                {
                        this.__defineLintWarningTestClass();
                        var demo = new demoapp.test.DemoTestClass();
                        demo.saySomething();
                }
        }
});
Reply | Threaded
Open this post in threaded view
|

Re: @lint ignoreUndefined(...) broken?

Thomas Herchenroeder

On 09/02/2013 10:18 AM, Florin Jurcovici wrote:

> Sorry for answering that late, I just got other stuff on my hands and forgot
> about this, I just stumbled upon the problem again, and Google was smart and
> led me to my own post.
>
> Here's a test that reproduces the problem - it runs through, but lint
> complains about demoapp.test.DemoTestClass being undefined when generate
> lint:
>
> qx.Class.define("demoapp.test.DemoTest",
> {
> extend : qx.dev.unit.TestCase,
>
> members :
> {
> __defineLintWarningTestClass: function()
> {
> qx.Class.define("demoapp.test.DemoTestClass", {
> extend: qx.core.Object,
> members:
> {
> saySomething: function()
> {
> var message = this.classname + " instance is doing well.";
> this.debug(message);
> console.log(message);
> }
> }
> });
> },
>
> /*
> * @lint ingoreUndefined(demoapp.test.DemoTestClass)
> * @lint ingoreUndefined(DemoTestClass)
> */

JSDoc comments need to positively start with '/** ' (ie.
slash-star-star-blank). If not they are not recognized as such [1][2].

T.

[1]
http://manual.qooxdoo.org/3.0/pages/development/api_jsdoc_ref.html#overall-jsdoc-structure
[2] http://usejsdoc.org/about-getting-started.html


------------------------------------------------------------------------------
Learn the latest--Visual Studio 2012, SharePoint 2013, SQL 2012, more!
Discover the easy way to master current and previous Microsoft technologies
and advance your career. Get an incredible 1,500+ hours of step-by-step
tutorial videos with LearnDevNow. Subscribe today and save!
http://pubads.g.doubleclick.net/gampad/clk?id=58040911&iu=/4140/ostg.clktrk
_______________________________________________
qooxdoo-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel