I finally had some quiet time this weekend and was attempting to work on issue #1995 when I realized that I don't really know what the coding policy is regarding line terminators.
The http://chirp.danplanet.com/projects/chirp/wiki/DevelopersProcess page specifically requests that changing whitespace and other stuff not be done unless it is within the scope of the issue being fixed--a perfectly reasonable request.
However, I noticed that most of the files under stock_configs seem to have CRLF line terminators while some of them have LF line terminators. Which one is preferred? I would prefer LF as this is what all the Python source files seem to use.
As I am about to reorder pretty much all 61 lines of the file stock_configs/Marine\ VHF\ Channels.csv, I figured I might as well ask here first before finishing up the patch. I feel that fixing the line terminators here for just this one file is probably acceptable in this instance.
Once I know for sure, I shall be raising another bug/ticket/issue/whatever to give all the rest of the stock_configs files the same line terminators.
Thanks and 73,
Tyler VA3DGN