[
https://issues.apache.org/jira/browse/OFBIZ-9870?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16287400#comment-16287400 ]
Jacques Le Roux commented on OFBIZ-9870:
----------------------------------------
Then you should say, we should RE-discuss this. It has been discussed in the past and the consensus is that we don't remove trailing spaces in patches because it hurts reviewers with false changes. So please if you want this dicussed start a thread in dev ML.
bq. You can adjust your IDE/Eclipse to ignore whitepaces in diffs, so you only see the real changes. That works very good.
I'm reviewing directly in my email clients, so this is not relevant for me. Maybe a solution would be to suggest another reviewing process, THAT can be discussed I think. And then possibly removing trailing spaces would be a good thing indeed.
> [Refactoring] Package org.apache.ofbiz.accounting.invoice
> ---------------------------------------------------------
>
> Key: OFBIZ-9870
> URL:
https://issues.apache.org/jira/browse/OFBIZ-9870> Project: OFBiz
> Issue Type: Sub-task
> Components: accounting
> Affects Versions: Trunk
> Reporter: Dennis Balkir
> Assignee: Michael Brohl
> Priority: Minor
> Fix For: Upcoming Release
>
> Attachments: OFBIZ-9870_org.apache.ofbiz.accounting.invoice.InvoiceServices_refactoring.patch, OFBIZ-9870_org.apache.ofbiz.accounting.invoice.InvoiceWorker_refactoring.patch
>
>
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
Locked
