Opened 2 years ago

Last modified 9 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 2 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 2 years ago by Paradoxon

Has a Patch: set

comment:2 Changed 2 years ago by diver

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

comment:3 Changed 2 years ago by korli

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 Changed 9 months ago by pulkomandy

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

comment:5 Changed 9 months ago by pulkomandy

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