Opened 4 years ago
Closed 18 months ago
#61181 closed enhancement (fixed)
LittleCMS 2 lcms2 - Please add option to disable register variable (or remove entirely)
| Reported by: | MrB74 | Owned by: | mascguy (Christopher Nielsen) |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | |
| Keywords: | Cc: | ||
| Port: | lcms2 |
Description (last modified by MrB74)
C++ standards post C++11 do not support register keywords. LittleCMS 2 introduced the CMS_NO_REGISTER_KEYWORD
The register keyword LittleCMS uses and requires C99, and this standard allows the use of ́register ́ keyword. Some compilers complain about this as being deprecated. Well, standards cannot be changed in retro- active way unless you have a time machine! But anyway, if you want to compile the code as C++14 or things like that, you can use the toggle CMS_NO_REGISTER_KEYWORD
Would it be possible to get an option to disable it, or just disabled it by default by setting uncommenting the #define CMS_NO_REGISTER_KEYWORD 1 in lcms2.h ?
thanks Craig
Change History (7)
comment:1 Changed 4 years ago by MrB74
| Description: | modified (diff) |
|---|
comment:2 Changed 4 years ago by MrB74
| Description: | modified (diff) |
|---|---|
| Summary: | LittleCMS lcms - Please add option to disable register variable (or remove entirely) → LittleCMS 2 lcms2 - Please add option to disable register variable (or remove entirely) |
comment:3 Changed 4 years ago by mf2k (Frank Schima)
comment:4 follow-up: 5 Changed 4 years ago by mf2k (Frank Schima)
| Port: | lcms2 added |
|---|
comment:6 Changed 18 months ago by mascguy (Christopher Nielsen)
| Owner: | set to mascguy |
|---|---|
| Status: | new → assigned |
comment:7 Changed 18 months ago by mascguy (Christopher Nielsen)
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
In the future, please fill in the Port field.