Opened 17 years ago
Last modified 16 years ago
#243 new Enhancement
Improve const-correctness
| Reported by: | elfring | Owned by: | davea |
|---|---|---|---|
| Priority: | Minor | Milestone: | Undetermined |
| Component: | Undetermined | Version: | |
| Keywords: | const patch | Cc: |
Description
Would you like to apply the advices from a well-known article to more places in your source files?
I suggest to add the key word "const" to the type specifiers for parameters like the following.
- string (functions "txfRenderString", "txfRenderFancyString")
- name (function "read_tga_texture")
- dir (function "txf_load_fonts")
- s (function "txf_render_string")
- N, label (global variables)
Attachments (3)
Change History (8)
comment:1 Changed 17 years ago by
| Component: | BOINC - API → Undetermined |
|---|---|
| Owner: | Bruce Allen deleted |
comment:2 Changed 17 years ago by
Another example:
The icon "usage.xpm" for the client GUI was tagged as read-only already. Would you also like to apply this property to more pictures to avoid messages like the following?
res/warning.xpm:63: Warnung: deprecated conversion from string constant to »char*«
comment:3 Changed 16 years ago by
| Keywords: | const-correctness read-only data structures added |
|---|
Would you like to reduce more warnings that I get during compilation by GCC 4.3.1?
comment:4 Changed 16 years ago by
| Keywords: | const patch added; const-correctness read-only data structures removed |
|---|
comment:5 Changed 16 years ago by
| Milestone: | 6.6 → Undetermined |
|---|---|
| Owner: | set to davea |
Note: See
TracTickets for help on using
tickets.
Removing component, since this could apply to the whole BOINC codebase.