On 09/10/2018 2:15 am, Allan McRae wrote: > On 9/10/18 11:05 am, morganamilo wrote: >> static struct pkg_operations local_pkg_ops = { >> - .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, >> - .get_packager = _cache_get_packager, > <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. > > A >
I could split adding the fields and the formatting to two patches if you prefer? Easier to read diffs and the struct stays pretty.
signature.asc
Description: OpenPGP digital signature