Yes, I agree. It's better to avoid using this unless one really has to.
Best,
Ruxi
On 04/17/2017 06:14 PM, David Case wrote:
> On Mon, Apr 17, 2017, Ruxi Qi wrote:
>
>> FYI, trailing whitespace can be easily eliminated by the sed in-place
>> substitution as follows, say to process all .F90 files in batch, do:
>>
>> sed -i "s/\s\+$//g" *.F90
> True, but let's not do this just for the sake of doing it, or because
> some integration script or compiler complains about trailing whitespace.
> Large-scale changes like this can make it impossible to track true code
> changes, and for really minimal gain.
>
> Even with Dave's changes, some which were made to satisfy a rather arbitrary
> 96-character line limit, have to balance the pain of having some
> long lines with the pain of not being able to compare code to earlier versions.
>
> Establishing consistent indentation at the beginning of lines (to make
> code flow and logic clearer) is probably worth the pain in most cases.
> Removing trailing whitespace is often not worth the pain.
>
> ....dac
>
>
> _______________________________________________
> AMBER-Developers mailing list
> AMBER-Developers.ambermd.org
> http://lists.ambermd.org/mailman/listinfo/amber-developers
_______________________________________________
AMBER-Developers mailing list
AMBER-Developers.ambermd.org
http://lists.ambermd.org/mailman/listinfo/amber-developers
Received on Mon Apr 17 2017 - 19:30:02 PDT
