bugGNU Octave - Bugs: bug #54206, Compilation warning on xerbla.c on...

 
 

bug #54206: Compilation warning on xerbla.c on stable branch

Submitted by:  Rik <rik5>
Submitted on:  Fri 29 Jun 2018 06:55:01 PM UTC  
 
Category:  Configuration and Build System Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Other
Status:  Fixed Assigned to:  John W. Eaton <jwe>
Originator Name:  Open/Closed:  Closed
Release:  4.4.0 Operating System:  GNU/Linux

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

( Jump to the original submission)

Fri 29 Jun 2018 09:21:05 PM UTC, comment #6:

LGMT, built with Clang 7 and GCC 8.

Mike Miller <mtmiller>
Project Administrator
Fri 29 Jun 2018 09:17:06 PM UTC, comment #5:

It was mean to make jwe do this change when he didn't have a compiler that was producing the warning and could give feedback. I changed the format code back to %ld for the F77_INT which is cast to long in the printf statement (http://hg.savannah.gnu.org/hgweb/octave/rev/194417044d1a).

Marking as fixed and closing report.

Rik <rik5>
Project Administrator
Fri 29 Jun 2018 09:00:45 PM UTC, comment #4:

No, I now get this warning with Clang 7

I think you shouldn't have dropped the '%ld', the warning message was about the '*' in the '%.*s' format conversion.

Mike Miller <mtmiller>
Project Administrator
Fri 29 Jun 2018 08:49:50 PM UTC, comment #3:

I pushed this change on stable and merged with default:

http://hg.savannah.gnu.org/hgweb/octave/rev/c05adc188b98

Is that OK?

John W. Eaton <jwe>
Project AdministratorIn charge of this item.
Fri 29 Jun 2018 07:54:58 PM UTC, comment #2:

I guess I was a little too smart using size_t here. This string won't be long (usually less than 9 characters), so it is probably fine to use int for the value.

John W. Eaton <jwe>
Project AdministratorIn charge of this item.
Fri 29 Jun 2018 07:34:49 PM UTC, comment #1:

Neat, I get this warning building with Clang 7

but no warning at all with GCC 7.

Mike Miller <mtmiller>
Project Administrator
Fri 29 Jun 2018 06:55:01 PM UTC, original submission:

While compiling stable with

I get the following compilation warning.

Seems easy enough to fix, and should be done before 4.4.1 release.

Rik <rik5>
Project Administrator

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by jwe (Posted a comment)
  • -email is unavailable- added by mtmiller (Posted a comment)
  • -email is unavailable- added by rik5 (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.

     

     

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-06-29 rik5 StatusConfirmed => Fixed
        Open/ClosedOpen => Closed
    2018-06-29 mtmiller StatusNone => Confirmed

    Back to the top


    Powered by Savane 3.3