Rene Scharfe <[email protected]> writes: > If format_tracking_info() returns 0 only if it didn't touch its strbuf > parameter, so it's OK to exit early in that case. Clean up sb in the > other case.
These two "if"s confuse me; perhaps the first one is not needed?

