| | [ Return to Bugs & Features | Post Text | Post File | SVN ⇄ GIT | Prev | Next ]
STR #3100
| Application: | FLTK Library |
|---|
| Status: | 5 - New |
|---|
| Priority: | 1 - Request for Enhancement, e.g. asking for a feature |
|---|
| Scope: | 3 - Applies to all machines and operating systems |
|---|
| Subsystem: | Build Files |
|---|
| Summary: | [patch] Allow printing to be disabled |
|---|
| Version: | 1.4-feature |
|---|
| Created By: | cand |
|---|
| Assigned To: | Unassigned |
|---|
| Fix Version: | 1.4.0 (SVN: v12131) |
|---|
| Update Notification: | |
|---|
Trouble Report Files:
[ Post File ]
Trouble Report Comments:
[ Post Text ]
| |
| #1 | cand
06:48 Jun 30, 2014 |
| Printing contributes significant size in the shared library. This patch lets it be disabled, saving 185kb off libfltk.so. | |
| |
| #2 | greg.ercolano
07:49 Jul 15, 2014 |
| Agreed, printing support seems currently intertwined in very low level
code, and pulls in a lot of overhead that many apps might not need.
I see it's implemented here as a compile time option, but perhaps
we can make it runtime configurable, e.g. w/func ptrs, or some such.
(I can envision projects that want to build multiple FLTK apps,
some might need printing, some might not) | |
| |
| #3 | cand
05:43 Aug 21, 2014 |
| Hm, how about a separate libfltk_print? Just like forms and images. | |
| |
| #4 | AlbrechtS
05:45 Sep 01, 2016 |
| I believe that printing can be disabled in branch-1.3-porting.
Since this won't be fixed in 1.3.x, this renders this STR useless (resolved), IMHO.
Can we close this STR? | |
| |
| #5 | cand
06:55 Sep 01, 2016 |
| Why not move the target to 1.4, until it's confirmed it can be disabled? | |
| |
| #6 | AlbrechtS
16:01 Sep 01, 2016 |
| No problem, bumped version from 1.3 to 1.4-feature, and changed priority from 2 (low) to 1 (RFE).
Note that the posted patch (fltk-no-printing.diff) was against FLTK 1.3.x and that the internals have changed a lot in FLTK 1.4.
I can see the CMake option "OPTION_PRINT_SUPPORT:BOOL=ON" that can be turned OFF to disable print support, but I can't tell how (if) this works on different platforms. I didn't find a configure option, but I didn't search hard. The above mentioned patch seems to provide a configure option, so this might complete the solution.
More investigation needed... | |
| |
| #7 | AlbrechtS
05:02 Nov 02, 2016 |
| Lauri, can we close this STR?
Does the CMake option work for you?
Do we still need an equivalent configure option? | |
| |
| #8 | cand
05:14 Nov 02, 2016 |
| I can't test anything in over a month. In general all config options are needed in both systems. | |
| |
| #9 | AlbrechtS
03:46 Dec 02, 2016 |
| Fixed in Subversion repository.
Thanks to Manolo for fixing this issue in svn r12131.
Lauri, please confirm that this works / is sufficient for you, and if so, please close this STR. Thanks. | |
| |
| #10 | cand
04:10 Dec 02, 2016 |
| Still going to be a while, I'm afraid. | |
| |
| #11 | AlbrechtS
06:14 Jun 25, 2018 |
| Just a reminder: can we now close this STR? | |
[ Return to Bugs & Features | Post Text | Post File ]
|
| |
