[PATCH v2 1/2] posix: Add compat glob symbol to not follow dangling symbols

[Adhemerval Zanella]

On 22/09/2017 18:43, Paul Eggert wrote:
> On 09/22/2017 06:02 AM, Adhemerval Zanella wrote:
>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â  ((flags & (GLOB_ERR | GLOB_NOESCAPE | GLOB_ALTDIRFUNC))
>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â  | GLOB_NOSORT | GLOB_ONLYDIR),
>
> The indenting is not quite right here: the "|" should be under the 2nd
> "(" in the previous line, not under the 1st.
Ack.

>
>> +Â Â Â Â Â Â Â Â Â  result = __glob (onealt,
>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â  ((flags & ~(GLOB_NOCHECK | GLOB_NOMAGIC))
>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â  | GLOB_APPEND), errfunc, pglob);
>
> As long as we're changing indenting anyway, we should fix the
> indenting here to match the usual GNU style. Something like this:
>
> Â Â Â Â Â Â Â Â Â Â Â Â Â  result = __glob (onealt,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â  ((flags & ~(GLOB_NOCHECK | GLOB_NOMAGIC))
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â  | GLOB_APPEND),
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â  errfunc, pglob);

Ack.

>
> Other than these minor indenting things it looks OK. glibc's glob.c is
> wrongly indented elsewhere, but that can be fixed separately.
>
> Proposed corresponding gnulib patch attached; it would keep gnulib
> glob.c in sync with glibc's, except for whitespace and the https:
> thing. CC'ing this to bug-gnulib.
I will commit both patch shortly, thanks for the review.