On 09/22/2016 03:53 AM, Gordian Edenhofer wrote:
> I wouldn't assume that the user has done anything wrong, rather
> something with the system is wrong if one feels the need to 'backup'
> packages.
Awesome, we must be in agreement, because I don't think people should
feel the need to backup
On Wed, 2016-09-21 at 23:28 -0400, Eli Schwartz wrote:
> On 09/21/2016 05:56 AM, Gordian Edenhofer wrote:
> >
> > You bar for insanity is whether the debian folks do it? Please
> > explain
> > yourself why do you think this is the wrong approach for the
> > problem.
> > My reasoning would be that
Making it size_t matches the return value of alpm_list_count() and
avoids the implicit cast to int.
Signed-off-by: Rikard Falkeborn
---
lib/libalpm/be_sync.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/lib/libalpm/be_sync.c
Since the number of packages is not used anywhere, just return a
boolean to avoid the implicit cast from size_t to int in be_local.c.
Use 0 as success to be consistent with db_validate.
Signed-off-by: Rikard Falkeborn
---
This patch is made on top of the "Fix Gcc