Andrew Gierth <andrew(at)tao11(dot)riddles(dot)org(dot)uk> writes:
> "Rikard" == Rikard Falkeborn <rikard(dot)falkeborn(at)gmail(dot)com> writes:
> Rikard> if (strcmp(*nodename, "+") == 0 ||
> Rikard> strcmp(*nodename, "-")) // <-- notice the lack of comparisson here
> Rikard> Should the second part of the || be strcmp(*nodename, "-") == 0?
> Yes it should.
Indeed. Considering how much I hate using strcmp's result as a boolean,
you'd think I'd have got that right. Thanks for noticing!
regards, tom lane