Hi, Bob fixed the most obvious, here are the outstanding warnings for dead code :
Description Resource Path Location Type Dead code JmolApp.java /Jmol/src/org/openscience/jmol/app line 372 Java Problem Dead code Jvxl.java /Jmol/src/org/openscience/jvxl line 283 Java Problem Dead code Jvxl.java /Jmol/src/org/openscience/jvxl line 313 Java Problem Dead code Mesh.java /Jmol/src/org/jmol/shape line 241 Java Problem Dead code ScriptMathProcessor.java /Jmol/src/org/jmol/script line 196 Java Problem Dead code ScriptMathProcessor.java /Jmol/src/org/jmol/script line 205 Java Problem Dead code SimplePopup.java /Jmol/src/org/jmol/popup line 400 Java Problem Nico On Tue, Aug 17, 2010 at 11:44 PM, Jonathan Gutow <gu...@uwosh.edu> wrote: > Nico, > Can you point to a specific piece of offending code? If the statements > really are the syntax you are suggesting they make little sense. > > Jonathan > On Aug 17, 2010, at 12:12 PM, Nicolas Vervelle wrote: > > Hi, > > Could we get rid of the 'dead code' warnings ? > Most of them are due to code like : if (true || xxx) ... > which makes xxx a dead code. > > If it's intended, I see 2 ways of removing the warning : > > - Removing the dead code (if the true is definitive, it will clear it > definitely) > - Replacing the 'true' by a constant (final static boolean) set to true > : it will be a lot more comprehensible and the compiler should stop to > complain :) > > Nico > > > ------------------------------------------------------------------------------ > This SF.net email is sponsored by > > Make an app they can't live without > Enter the BlackBerry Developer Challenge > http://p.sf.net/sfu/RIM-dev2dev > _______________________________________________ > Jmol-developers mailing list > Jmol-developers@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/jmol-developers > > > Dr. Jonathan H. Gutow > Chemistry Department gu...@uwosh.edu > UW-Oshkosh Office: 920-424-1326 > 800 Algoma Boulevard FAX:920-424-2042 > Oshkosh, WI 54901 > http://www.uwosh.edu/facstaff/gutow > > > > ------------------------------------------------------------------------------ > This SF.net email is sponsored by > > Make an app they can't live without > Enter the BlackBerry Developer Challenge > http://p.sf.net/sfu/RIM-dev2dev > _______________________________________________ > Jmol-developers mailing list > Jmol-developers@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/jmol-developers > >
------------------------------------------------------------------------------ This SF.net email is sponsored by Make an app they can't live without Enter the BlackBerry Developer Challenge http://p.sf.net/sfu/RIM-dev2dev
_______________________________________________ Jmol-developers mailing list Jmol-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/jmol-developers