On 10/8/18 9:15 PM, Allan McRae wrote: > On 9/10/18 11:05 am, morganamilo wrote: > <snip> >> + .get_base = _cache_get_base, >> + .get_desc = _cache_get_desc, >> + .get_url = _cache_get_url, >> + .get_builddate = _cache_get_builddate, >> + .get_installdate = _cache_get_installdate, > ... > > Is there any objection to killing this formatting rather than having a > lot of uselessness in the patch.
I'm rather meh about keeping things aligned like this in the first place, so I'm okay with killing the formatting. I mean, indenting the beginning of the line makes sense. Making variable assignments line up is useless IMO. But if you will do it then you should leave space for it to grow... -- Eli Schwartz Bug Wrangler and Trusted User
signature.asc
Description: OpenPGP digital signature
