This is the mail archive of the gdb-patches@sourceware.cygnus.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: [PATCH] Fix for target.c:do_target_signal_to_host()


Mark Kettenis wrote:
 
> While adding the comments I noticed that the function will return 32
> for TARGET_SIGNAL_REALTIME_32 even if it doesn't exist, and that (with
> my patch) for the other RT signals the upper bound was checked, but
> not the lower bound.  Not likely to be very problematic, but this is
> not right according to the comment at the top of the function.
> 
> I also moved the TARGET_SIGNAL_REALTIME_32 case within the
> default: where the other RT signals are handled.  I think this
> makes it easier to understand the code.

Yes.  Ok.

	Andrew

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]