libericstools: list: Remove tautologic comparison from boolean.

Since an `unsigned` variable is always non-negative, we don't need to test
it. If that failed, then we would have real trouble with the toolchain.

Also, while we are at it, remove some pesky trailing whitespaces from the
lines of the function.

Signed-off-by: Rogério Brito <rbrito@ime.usp.br>
bwmon-gargoyle: Document why an allocated FILE leak is OK.

If the operating system were not helpful, we would need to fix the code by
checking in separate the status of the input file and the status of the
output file and deallocating/closing them.

Signed-off-by: Rogério Brito <rbrito@ime.usp.br>
qos-gargoyle: Remove unused parameter for fprintf.

While we are at it, we also remove some trailing whitespace in the function.

Signed-off-by: Rogério Brito <rbrito@ime.usp.br>
ddns-gargoyle: Fix gross error with if with empty body.

We also remove pesky trailing whitespaces.

Signed-off-by: Rogério Brito <rbrito@ime.usp.br>
gargoyle: session_validator: Remove duplicate break from case.

We also clean some whitespace.

Signed-off-by: Rogério Brito <rbrito@ime.usp.br>
gargoyle: session_validator: Vastly simplify code.

While the code is smaller, it is also branch-free, which should help with
generation of the code.

Signed-off-by: Rogério Brito <rbrito@ime.usp.br>
gargoyle: session_validator: More simplification of the code.

This further reduces the memory consumption of variables on the stack and
also reduces by two the number of calls to `sprintf`.

Signed-off-by: Rogério Brito <rbrito@ime.usp.br>
fix bug preventing individual packages from installing correctly, introduced when we implemented handling of provides functionality