[ http://issues.apache.org/jira/browse/OFBIZ-525?page=comments#action_12455987 ] Stefan Huehner commented on OFBIZ-525: --------------------------------------
Hi Jacques, yes this patch addresses only part of the files containing unused variables. Just wanted to get the first bunch out. In addition i tried to leave some obvious/standard variables like 'delegator' in place. The whole patch (series) is of course optional. I personally find i better to remove these vars, but one could equally say to leave them in case of future need. Please if this is applicable. I'll put further work of this issue on hold until then. > Remove unused variables > ----------------------- > > Key: OFBIZ-525 > URL: http://issues.apache.org/jira/browse/OFBIZ-525 > Project: OFBiz (The Open for Business Project) > Issue Type: Improvement > Affects Versions: SVN trunk > Reporter: Stefan Huehner > Assigned To: Jacques Le Roux > Priority: Trivial > Attachments: ofbiz_unused_vars.diff > > > Attached patch removes variables which are only assigned but never read. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira
