Review: Needs Fixing
> 'git blame -w' to ignore whitespace change?
To my mind this is the important point. I'm fairly open about accepting the occasional reformatting which 'git diff --ignore-all-space' renders as empty. And I would be fairly closed to reformatting which doesn't. So eg. no moving brackets between lines. Also, this isn't something to be done frequently. (once per decade is ok)
This change is almost empty. 'git diff --ignore-all-space' still shows the removal of blank lines at the end of files. I'm not so bother this time, although I don't see much point to doing this in future.
As far as style rules. I'm not so strict on aspects other than indentation. There my first rule is to be consistent. With indentation of 4 spaces being a very strong preference (where allowed).
Beyond that, I'm not so concerned beyond a small bias towards schemes which qtcreator can apply automatically. In short, this is not an area where I want to spend much time.
So this change is acceptable to me, provided it doesn't complicate upmerge by whatever version of Git Andrew has these days.
I'm marking this "Needs Fixing" since Dirk has indicated he has more work to do.
--
https://code.launchpad.net/~dirk.zimoch/epics-base/+git/epics-base/+merge/382303
Your team EPICS Core Developers is subscribed to branch epics-base:7.0.
- References:
- [Merge] ~dirk.zimoch/epics-base:CleanupWhitespace into epics-base:7.0 Dirk Zimoch via Core-talk
- Navigate by Date:
- Prev:
Build failed: epics-base base-fix-epicsFindSymbol-463 AppVeyor via Core-talk
- Next:
Re: [Merge] ~bfrk/epics-base:address-modifiers into epics-base:7.0 Ben Franksen via Core-talk
- Index:
2002
2003
2004
2005
2006
2007
2008
2009
2010
2011
2012
2013
2014
2015
2016
2017
2018
2019
<2020>
2021
2022
2023
2024
- Navigate by Thread:
- Prev:
Re: [Merge] ~dirk.zimoch/epics-base:CleanupWhitespace into epics-base:7.0 Ben Franksen via Core-talk
- Next:
Re: [Merge] ~dirk.zimoch/epics-base:CleanupWhitespace into epics-base:7.0 Dirk Zimoch via Core-talk
- Index:
2002
2003
2004
2005
2006
2007
2008
2009
2010
2011
2012
2013
2014
2015
2016
2017
2018
2019
<2020>
2021
2022
2023
2024
|