geany/geany — comment on commit efcce8808e92ba56684605790959cb014a11c9c8
File:  line 0
By b4n:

Le 10/11/2011 18:17, Nick Treleaven a écrit :
> Any comments on utils_rebind()? I think 'rebind' looks nicer than _set_and_free or _replace.
> Rebind isn't obvious, but is shorter than utils_set_and_free and clearer than utils_replace.

Why not, though I think "setptr" was clearer. I've no real opinion here.

>> setptr should definitely at least expand to `do { … } while (0)`
>> rather than `{ … }`, gave me one or two headaches.
>
> I accept this is best practice and we should do it, but I'm curious what the headaches were?

IIRC, I did write:

if (...)
setptr(...);
else
...

and wondered a little why GCC complained about the "else" not having a
matching "if".