Opened 8 years ago
Last modified 6 years ago
#13269 new enhancement
Add BStringList support for CodyCam Settings
Reported by: | Paradoxon | Owned by: | mmu_man |
---|---|---|---|
Priority: | normal | Milestone: | Unscheduled |
Component: | Applications/CodyCam | Version: | R1/Development |
Keywords: | Cc: | ||
Blocked By: | Blocking: | ||
Platform: | All |
Description
Improve Setting API from CodyCam for future general use of this API.
Attachments (1)
Change History (6)
by , 8 years ago
Attachment: | 0005-Added-a-stringList-support-for-settingfiles.patch added |
---|
comment:1 by , 8 years ago
patch: | 0 → 1 |
---|
comment:2 by , 8 years ago
Component: | - General → Applications/CodyCam |
---|---|
Owner: | changed from | to
comment:3 by , 8 years ago
comment:5 by , 6 years ago
patch: | 1 → 0 |
---|
Note:
See TracTickets
for help on using tickets.
Please fix the code style: space before and after '=' or '>' or '!=', after ';' in a for loop, check for NULL instead of '!', '{' on the same line as the preceding while.
I'm not sure the destructor works correctly in the basic construction/destruction sequence, as both fDefaultStringList and fStringList point to the same object.