On Jan 28, 2010, at 10:13 AM, hans...@apache.org wrote:

> 
> Added: 
> ofbiz/trunk/specialpurpose/ebay/src/org/ofbiz/ebay/EbayStoreInventoryServices.java
> URL: 
> http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ebay/src/org/ofbiz/ebay/EbayStoreInventoryServices.java?rev=904006&view=auto
> ==

This is a minor remark but I have noticed that the formatting of the Java file 
above is not 100% inline with our code formatting conventions; for example:

if(condition){
}else{
}

instead of

if (condition) {
} else {
}

and also

if (condition == false) 

instead of:

if (!condition)

Kind regards,

Jacopo

Reply via email to