When getting info about heap usage (e.g. ld --stats) I think it is desiderable to have something more fine-grained than sbrk() on systems where the default allocator is jemalloc (namely, FreeBSD). I wasn't sure who to put as reviewer to this change, feel free to comment. Also, I noticed that GetMallocUsage() isn't unit-tested (or at least I didn't find where it is unit-tested), so, maybe , enhancing the testing coverage might be considered in this (or a subsequent) commit.
I do agree with you and I'm generally a big fan of avoiding 'else' unless necessary but a couple of lines below that if/else is used so I'd keep that for consistency and (eventually) change in a subsequent commit, in order to not mix style/functional modifications.
Does this make sense to you?