If we are going to switch to this construction (I don't feel strongly either way, but it seems like you and @carlosmn have come to an agreement), then the error message should be set inside `git__page_size` when it occurs (so all callers get the same message), and this code can just be:

```c
error = git__page_size(&page_size);
if (error < 0)
    return error;
```