FLTK logo

Re: [RFE] STR #2728: Add test program to test for shadowed variables (-Wshadow)

FLTK matrix user chat room
(using Element browser app)   FLTK gitter user chat room   GitHub FLTK Project   FLTK News RSS Feed  
  FLTK Apps      FLTK Library      Forums      Links     Login 
 All Forums  |  Back to fltk.bugs  ]
 
Previous Message ]New Message | Reply ]Next Message ]

Re: [RFE] STR #2728: Add test program to test for shadowed variables (-Wshadow) Albrecht Schlosser 08:42 Apr 18  
 
[STR Closed w/Resolution]

Link:         https://www.fltk.org/str.php?L2728
Version:      1.3-feature
Fix Version:  1.4.0
Git Commit:   3fc8875756c3ce02b1b45bd0383a0c307c442dde
Assigned To:  AlbrechtS

Fixed in Git repository.

Fixed in commit bb45198413ef8efe236afbd665a4623239ac0da0
and updated in  3fc8875756c3ce02b1b45bd0383a0c307c442dde.

I decided to add my own test/shadow_variables.cxx as a new test program
only in CMake builds. This should be sufficient to detect warnings early. I
deliberately decided not to add this to configure/make builds because that
will be dropped in 1.5 anyway.

The header file 'include_all.h' is generated by CMake dynamically during
the CMake configure phase. There's no shell involved (no perl, no awk
etc.).

Side note: I detected two minor issues with this new feature and fixed them
in commit 2c21e520f4859904739deb7d058b084f314ed47d *before* I added this
new feature.

Thanks to Greg for the inspiration.


Link:         https://www.fltk.org/str.php?L2728

Direct Link to Message ]
 
     
Previous Message ]New Message | Reply ]Next Message ]
 
 

Comments are owned by the poster. All other content is copyright 1998-2024 by Bill Spitzak and others. This project is hosted by The FLTK Team. Please report site problems to 'erco@seriss.com'.