Re: [xwiki-dev] XECLIPSE.R.1.0.M1

Hi Tharindu, I have done a first pass on the code review and I have a few comments: 1) You seem to have use Cloudgarden's Jigloo and unfortunately the license isn't good enough for us so we cannot use it... Here's what it says in the generated file: /** * This code was edited or generated using CloudGarden's Jigloo SWT/ Swing GUI Builder, which is free * for non-commercial use. If Jigloo is being used commercially (ie, by a corporation, company or * business for any purpose whatever) then you should purchase a license for each developer using * Jigloo. Please visit www.cloudgarden.com for details. Use of Jigloo implies acceptance of these * licensing terms. A COMMERCIAL LICENSE HAS NOT BEEN PURCHASED FOR THIS MACHINE, SO JIGLOO OR THIS * CODE CANNOT BE USED LEGALLY FOR ANY CORPORATE OR COMMERCIAL PURPOSE. */ We cannot use Jigloo because we want commercial companies using XWiki under the LGPL license. Thus we cannot commit this code. Could you find some alternative? The UI doesn't seem too complex so I should be easy to do it by hand I think. 2) Thanks for your javadoc. It's nice and helps a lot. However it can be improved, especially for Classes. This is the most important Javadoc and should answer the question "what is the class for?". Also please see: http://www.xwiki.org/xwiki/bin/view/Community/ CodeStyle#HDonotuse27public27ininterfaces http://www.xwiki.org/xwiki/bin/view/Community/ CodeStyle#HDonotduplicatemethodcommentswithparameterscomments Parameters and return value shouldn't start with "-" in javadoc. See the Sun's Javadoc coding conventions. 3) Please add copyright to all classes and don't use the @author tag. Instead we'll add Asiri and yourself to the Hall of Fame + in the SVN commits. The rest sounds good. We need to add maven build of course. I'll start looking into that after we fix the points above. Also, I haven't coded plugins in Eclipse for a very very long time so I have no idea if what you've done is the right way of doing it so I'll assume it is :-) Thanks again and keep up the good work! -Vincent On Jul 15, 2007, at 10:04 AM, tharindu jayasuriya wrote: