|
|
commit 2392589f052e29803cf4efc57c02ecfa7a7da2f1
Author: Albrecht Schlosser <albrechts.fltk@online.de>
AuthorDate: Fri Sep 18 13:39:04 2020 +0200
Commit: Albrecht Schlosser <albrechts.fltk@online.de>
CommitDate: Fri Sep 18 13:41:16 2020 +0200
Return 1 from Fl_Menu_Item::value() for checked items
Previously (FLTK 1.3.x) the return value of a checked menu item
was FL_MENU_VALUE (4) but the docs warned explicitly:
"You should not rely on a particular value, only zero or non-zero."
FL/Fl_Menu_Item.H | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git FL/Fl_Menu_Item.H FL/Fl_Menu_Item.H
index 65be443..f4b7500 100644
--- FL/Fl_Menu_Item.H
+++ FL/Fl_Menu_Item.H
@@ -304,13 +304,13 @@ struct FL_EXPORT Fl_Menu_Item {
*/
int radio() const {return flags&FL_MENU_RADIO;}
/** Returns the current value of the check or radio item.
- This is zero (0) if the menu item is not checked and
- non-zero otherwise. You should not rely on a particular value,
- only zero or non-zero.
- \note The returned value for a checked menu item as of FLTK 1.3.2
- is FL_MENU_VALUE (4), but may be 1 in a future version.
+ This is zero (0) if the menu item is not checked and non-zero otherwise.
+ \since 1.4.0 this method returns 1 if the item is checked but you
+ should not rely on a particular value, only zero or non-zero.
+ \note The returned value for a checked menu item was FL_MENU_VALUE (4)
+ before FLTK 1.4.0.
*/
- int value() const {return flags&FL_MENU_VALUE;}
+ int value() const {return (flags & FL_MENU_VALUE) ? 1 : 0;}
/**
Turns the check or radio item "on" for the menu item. Note that this
does not turn off any adjacent radio items like set_only() does.
[ Direct Link to Message ] | |
|
| |