To me it seems like a wasted effort, what do we gain by removing warnings and adding FIXMEs? It seems like all we get are unneeded comments and a few more chunks to review.
But please don't take my comments as criticism, I appreciate the effort you're putting in and have spent many hours doing similar work myself over the years. Regards Scott HotWax Media http://www.hotwaxmedia.com On 9/02/2011, at 12:52 AM, mrisal...@libero.it wrote: > What I have to do it in this case leave the compilation warning or remove it > putting a //FIXME ? > > I would like to before to remove all the compilation warning and then we > could > work on the //FIXME later. > > What did you think of it ? > > Thanks > Marco > >> ----Messaggio originale---- >> Da: jacques.le.r...@les7arts.com >> Data: 08/02/2011 8.29 >> A: <dev@ofbiz.apache.org> >> Ogg: Re: svn commit: r1068120 - > /ofbiz/trunk/framework/base/src/org/ofbiz/base/config/JNDIConfigUtil.java >> >> Maybe there is a reason? Because I saw Marco fixing a lot of them... >> >> Jacques >> >> From: "Adrian Crum" <adrian.c...@sandglass-software.com> >>> It would be nice if we fixed the warnings instead of just hiding them. >>> >>> -Adrian >>> >>> On 2/7/2011 12:46 PM, mrisal...@apache.org wrote: >>>> Author: mrisaliti >>>> Date: Mon Feb 7 20:46:53 2011 >>>> New Revision: 1068120 >>>> >>>> URL: http://svn.apache.org/viewvc?rev=1068120&view=rev >>>> Log: >>>> Remove compilation warnings of an unused private method in JNDIConfigUtil > (OFBIZ-4102) >>>> >>>> Modified: >>>> ofbiz/trunk/framework/base/src/org/ofbiz/base/config/JNDIConfigUtil. > java >>>> >>>> Modified: > ofbiz/trunk/framework/base/src/org/ofbiz/base/config/JNDIConfigUtil.java >>>> URL: >>>> http://svn.apache. > org/viewvc/ofbiz/trunk/framework/base/src/org/ofbiz/base/config/JNDIConfigUtil. > java?rev=1068120&r1=1068119&r2=1068120&view=diff >>>> > ============================================================================== >>>> --- ofbiz/trunk/framework/base/src/org/ofbiz/base/config/JNDIConfigUtil. > java (original) >>>> +++ ofbiz/trunk/framework/base/src/org/ofbiz/base/config/JNDIConfigUtil. > java Mon Feb 7 20:46:53 2011 >>>> @@ -43,6 +43,7 @@ public class JNDIConfigUtil { >>>> } >>>> } >>>> >>>> + @SuppressWarnings("unused") >>>> private static Document getXmlDocument() throws > GenericConfigException { >>>> try { >>>> return ResourceLoader.getXmlDocument(JNDIConfigUtil. > JNDI_CONFIG_XML_FILENAME); >>>> >>>> >>> >> >> >> > >
smime.p7s
Description: S/MIME cryptographic signature