[PATCH] [readline] Fix compilation on MinGW

Tom Tromey tom@tromey.com
Tue Aug 13 19:46:00 GMT 2019


>>>>> "Christian" == Christian Biesinger via gdb-patches <gdb-patches@sourceware.org> writes:

Christian> S_ISLNK should not be called outside of an #ifdef.
Christian> However, this specific call is actually unnecessary, because
Christian> linkok can only be 0 if S_ISLNK is true, per the code above.

Christian> I have sent a bug report to bug-readline, though it does not show
Christian> up in the online archive yet.

I didn't see it show up there.  I wonder if something went wrong.

Christian> 2019-08-13  Christian Biesinger  <cbiesinger@google.com>

Christian> 	* colors.c (_rl_print_color_indicator): Remove unnecessary
Christian> 	S_ISLNK check to fix compilation on MinGW.

This looks fine to me.  Please check it in.

It's important to be sure to follow up with upstream readline; and to
update the patch in the local tree if Chet uses a different patch.

thank you,
Tom



More information about the Gdb-patches mailing list