Opened 3 years ago

Last modified 16 months 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:
Has a Patch: no Platform: All

Description

Improve Setting API from CodyCam for future general use of this API.

Attachments (1)

0005-Added-a-stringList-support-for-settingfiles.patch (3.1 KB ) - added by Paradoxon 3 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 by Paradoxon, 3 years ago

Has a Patch: set

comment:2 by diver, 3 years ago

Component: - GeneralApplications/CodyCam
Owner: changed from nobody to mmu_man

comment:3 by korli, 3 years ago

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.

comment:4 by pulkomandy, 16 months ago

Patch migrated to Gerrit: https://review.haiku-os.org/437

comment:5 by pulkomandy, 16 months ago

Has a Patch: unset
Note: See TracTickets for help on using tickets.