[tz] indentation in localtime.c

Steffen Nurpmeso steffen at sdaoden.eu
Thu Mar 31 16:53:57 UTC 2022


Paul Eggert wrote in
 <430bc674-ecf4-877b-31b9-5e469dcc7b77 at cs.ucla.edu>:
 |On 3/25/22 09:46, Ian Abbott via tz wrote:
 |> 
 |> I'm not sure, but from the "git blame" output, it looks like Arthur 
 |> Olson pretty much stuck with a shift width of 8 (matching the tab size), 
 |> and Paul's shift widths were a bit more variable, perhaps to fit the 
 |> code in 80 columns nicely, and also more use of lining up the parts of 
 |> expressions split across lines with the opening parentheses of the 
 |> previous lines.
 |
 |Yes, Arthur and I have different indentation styles. I've found lesser 
 |indentation to be useful when squeezing in fixes that require deeper 
 |nesting, without going past 80 columns or altering lines that don't need 
 |changing.
 |
 |There is a tension between having source code that looks nice and 
 |consistent now, vs a repository where "git blame" works all the way back 
 |to the beginning.

I am not sure whether -w could overcome this completely.  I have

  alias.bla blame -w -CCC -M4

 --End of <430bc674-ecf4-877b-31b9-5e469dcc7b77 at cs.ucla.edu>

--steffen
|
|Der Kragenbaer,                The moon bear,
|der holt sich munter           he cheerfully and one by one
|einen nach dem anderen runter  wa.ks himself off
|(By Robert Gernhardt)



More information about the tz mailing list