DO NOT REPLY TO THIS MESSAGE.  INSTEAD, POST ANY RESPONSES TO THE LINK BELOW.

[STR New]

Link: http://www.fltk.org/str.php?L2714
Version: 1.4-feature


Matt: wait, are you sure it needs to be 1.4?

I think this str can be solved within the 1.3.x series
without variable name changes by simply moving source in the .H
for the methods he's cited from .H -> .cxx.

Note he's not trying to get the fltk lib to compile with -Wshadow,
just his own apps.

So that could just mean relocating code, without changing anything.
(I've already modified Fl_Tree's in r9085, and working on
Fl_Table. In my case I moved all the code, not just his methods,
but for other fltk files, small amounts of code could be moved)

I don't think we need to bump to 1.4 to fix this; no abi issues.
It can probably be held for 1.3.2, unless the plan is to go
for a 1.4.x asap. I figure there's still room for a few maintenance
releases for 1.3.

To test for shadow problems for apps, we could make one test program
that #include's all fltk widgets, instancing them once, and have just
that test program compile with -Wshadow. This way future builds
will note shadow errors in our .H's.


Link: http://www.fltk.org/str.php?L2714
Version: 1.4-feature

_______________________________________________
fltk-bugs mailing list
fltk-bugs@easysw.com
http://lists.easysw.com/mailman/listinfo/fltk-bugs

Reply via email to