On Thu, Dec 03, 2015 at 08:53:43PM +0000, Sebastian Kügler wrote: > On Thursday, December 03, 2015 12:42:20 Dirk Hohndel wrote: > > Do we really want this? It seems like a switch statement and we format > > them with case xxx: at the same indentation level as the switch (to save > > one level of indentation). This patch shows the problem with not doing > > that - a LOT of emptyness in the source file (i.e., very deep > > indentation). > > Yes, it's a list of actions (indicated by the [] around it, with a group in > it. The indentation is currently just not consistent (also inside the actions > it jumps around, resulting in weirdly placed brackets.
OK, I'll take the patch. This is of course a file I'm working on right now to do the split of preferences / cloud credentials and to make sure that the dive list is loaded after the credentials have been entered... > I'll look into splitting out the action into their own file in a separate > patch. OK. /D _______________________________________________ subsurface mailing list [email protected] http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface
