The first patch is needed for $subject, I figured I'd let 2/2 tag along, but as it itself notes it's just one of numerous warnings SunCC emits on our sources. Seems sensible to fix while we're at it though. Ævar Arnfjörð Bjarmason (2): strmap: do not "return" in a void function style: do not "break" in switch() after "return" apply.c | 2 -- builtin/fast-export.c | 1 - strmap.h | 4 ++-- 3 files changed, 2 insertions(+), 5 deletions(-) -- 2.29.2.222.g5d2a92d10f8