[pacman-dev] [PATCH v2] libalpm: parse {check, make}depends when reading database

Eli Schwartz eschwartz at archlinux.org
Tue Oct 9 01:20:07 UTC 2018


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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.archlinux.org/pipermail/pacman-dev/attachments/20181008/b7cca0cf/attachment.asc>


More information about the pacman-dev mailing list