#3791: JSLint errors in the SCAYT plugin
---------------------------------+------------------------------------------
  Reporter:  fredck              |       Owner:  garry.yao        
      Type:  Bug                 |      Status:  assigned         
  Priority:  Normal              |   Milestone:  CKEditor 3.0     
 Component:  UI : Spell Checker  |     Version:  SVN (CKEditor)   
Resolution:                      |    Keywords:  Confirmed Review?
---------------------------------+------------------------------------------

Comment(by fredck):

 Replying to [comment:5 garry.yao]:
 > Actually it's not a global variable writing/initialization violation,
 it's just global variable reading and not harmful

 Definitely there is no violation and it's not harmful. But this is bad
 coding practice.

 > we may consider turn off this option from our lint tools.

 No way! Proper variable declaration is one of the most important checks.
 Undeclared vars have less performance, are not compressed and may collide
 easily.

-- 
Ticket URL: <http://dev.fckeditor.net/ticket/3791#comment:6>
FCKeditor <http://www.fckeditor.net/>
The text editor for Internet
------------------------------------------------------------------------------
Enter the BlackBerry Developer Challenge  
This is your chance to win up to $100,000 in prizes! For a limited time, 
vendors submitting new applications to BlackBerry App World(TM) will have
the opportunity to enter the BlackBerry Developer Challenge. See full prize  
details at: http://p.sf.net/sfu/Challenge
_______________________________________________
FCKeditor-Trac mailing list
FCKeditor-Trac@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac

Reply via email to