René Scharfe <l.s.r@xxxxxx> writes: >> I think René meant this: >> >> /* just for side effects */ >> skip_prefix(name, "refs/heads/", &name) || >> skip_prefix(name, "refs/tags/", &name) || >> skip_prefix(name, "refs/remotes/", &name); >> >> return name; >> >> which still short-sircuits, even though I do think it looks >> strange; "correct but strange". > > Yes. At least to me it looks less strange than the same lines wrapped > in "if ... /* nothing */". Yup, after looking at it again, it does not look so "strange" to me. I probably should have said "unusual but correct", not "correct but strange".