Bruce Momjian wrote:
>*** 247,255 ****
> int len = strlen(path);
>
> if (len > 2 && strcmp(path + len - 2, "/.") == 0)
>- {
> trim_directory(path);
>- }
> else if (len > 3 && strcmp(path + len - 3, "/..") == 0)
> {
> trim_directory(path);
>
>
>
As a matter of style I hate this. Stripping the braces from a branch of
an if statement on the ground that it only has one statement, when other
branches might be multi-statement blocks is just downright ugly. I
thought the outcome of a recent discussion was that we would stop doing
that, at least automatically from pgindent, but I don't see any reason
to do it manually either.
cheers
andrew