[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kdevelop-bugs
Subject:    [Bug 298804] Context menu option 'Format files' is  dangerous
From:       Romek <roman.konieczny () gmail ! com>
Date:       2012-04-26 7:56:23
Message-ID: bug-298804-40295-m7shEdjhG8 () http ! bugs ! kde ! org/
[Download RAW message or body]

https://bugs.kde.org/show_bug.cgi?id=298804

--- Comment #2 from Romek <roman.konieczny@gmail.com> ---
Thank you for agreeing on warn dialog.

Unfortunately VCS cannot be always used easily to revert changes. In my case,
formatting options caused some code to be eg. surrounded by {}. Such change
cannot be easily filtered out by telling diff to ignore whitespace changes. So,
I would opt for making an option for creating a diff (patch) before executing
formatting, and then have an option to revert changes and apply the diff.  
However I understand this is quite complicated, and there are more important
things to be changed prior to this.

-- 
You are receiving this mail because:
You are the assignee for the bug.

_______________________________________________
KDevelop-bugs mailing list
KDevelop-bugs@kdevelop.org
https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-bugs
[prev in list] [next in list] [prev in thread] [next in thread]