Re: [libgfortran,patch] Silence a warning

2013-11-17 Thread Janne Blomqvist
On Sun, Nov 17, 2013 at 1:05 PM, FX wrote: > This attach patch adds an assert() in the library to fix PR 51828, i.e. > silence a “may be used uninitialized” warning. > > Built and regtested on x86_64-apple-darwin13. OK to commit? > > FX > Ok, thanks. -- Janne Blomqvist

[libgfortran,patch] Silence a warning

2013-11-17 Thread FX
This attach patch adds an assert() in the library to fix PR 51828, i.e. silence a “may be used uninitialized” warning. Built and regtested on x86_64-apple-darwin13. OK to commit? FX libwarning.ChangeLog Description: Binary data libwarning.diff Description: Binary data

Re: [libgfortran, patch] Silence a warning in libgfortran's runtime/error.c

2011-11-08 Thread Janne Blomqvist
On Tue, Nov 8, 2011 at 12:33, FX wrote: > This patch for PR 47972 uses __builtin_choose_expr instead of the current > if-else, avoiding the type warning for the branch not taken. This was > suggested by Jakub in the PR itself. > > Bootstrapped and regtested on x86_64-linux, OK to commit to trunk

[libgfortran, patch] Silence a warning in libgfortran's runtime/error.c

2011-11-08 Thread FX
This patch for PR 47972 uses __builtin_choose_expr instead of the current if-else, avoiding the type warning for the branch not taken. This was suggested by Jakub in the PR itself. Bootstrapped and regtested on x86_64-linux, OK to commit to trunk? FX typewarning.ChangeLog Description: Binary

Re: [libgfortran, patch] Silence a warning in fallback round() implementation

2011-11-08 Thread Tobias Burnus
On 11/08/2011 10:06 AM, FX wrote: PR 47970 is about a warning in libgfortran/intrinsics/c99_functions.c for platforms without round() and ceill(), because round() is used in roundl() before it's prototype is given. Moving the round() prototype and implementation earlier in the file fixes it.

[libgfortran, patch] Silence a warning in fallback round() implementation

2011-11-08 Thread FX
PR 47970 is about a warning in libgfortran/intrinsics/c99_functions.c for platforms without round() and ceill(), because round() is used in roundl() before it's prototype is given. Moving the round() prototype and implementation earlier in the file fixes it. Patch tested by manually editing my