| | [ Return to Bugs & Features | Roadmap 1.1 | SVN ⇄ GIT ]
STR #744
| Application: | FLTK Library |
|---|
| Status: | 1 - Closed w/Resolution |
|---|
| Priority: | 2 - Low, e.g. a documentation error or undocumented side-effect |
|---|
| Scope: | 3 - Applies to all machines and operating systems |
|---|
| Subsystem: | Documentation |
|---|
| Summary: | signature confusion Fl_Browser_::full_height(), etal. |
|---|
| Version: | 1.1.6 |
|---|
| Created By: | wilson.afn |
|---|
| Assigned To: | mike |
|---|
| Fix Version: | 1.1.7 (SVN: v4063) |
|---|
| Update Notification: | |
|---|
Trouble Report Files:
No files
Trouble Report Comments:
| |
| #1 | wilson.afn
06:24 Mar 03, 2005 |
| I spent TWO LOUSY DAYS poking the wrong place with GDB but refusing to consider my own overridden item_selected(...) with its 13-character function body. Finally, I discovered I had omitted the "const" keyword in my declaration/definition, and my method wasn't even being called! (Allah! C++ is a perfectly *awful* language!)
While Fl_Browser_::item_selected(void*) const **IS** properly documented, and I omitted "const" as the result of MY OWN STUPIDITY, I decided to check the other methods I tried to override.
I discovered ::full_height() , ::full_width(), ::item_quick_height(), ::incr_height(), ::item_width, ::item_height() and ::item_draw() are documented as unadorned by the miserable "const", but in Fl_Browser_.H, "const" has damnably wormed its way back into their declaration.
I had overridden full_height() and it seemed to be working, but injecting a printf() soon revealed that it wasn't even being called. (Strangely, I had somehow included the lousy "const" in my declaration of two other methods despite the fact it was omitted in the docs.)
| |
| |
| #2 | mike
11:18 Mar 05, 2005 |
| Fixed in Subversion repository.
| |
[ Return to Bugs & Features ]
|
| |
