Merge lp://staging/~thomas-voss/location-service/unify-settings-backend into lp://staging/location-service

Proposed by Thomas Voß
Status: Merged
Approved by: Thomas Voß
Approved revision: 310
Merged at revision: 310
Proposed branch: lp://staging/~thomas-voss/location-service/unify-settings-backend
Merge into: lp://staging/location-service
Diff against target: 821 lines (+199/-238)
19 files modified
src/location/CMakeLists.txt (+2/-0)
src/location/cmds/provider.cpp (+5/-3)
src/location/cmds/run.cpp (+1/-1)
src/location/provider_registry.cpp (+2/-14)
src/location/provider_registry.h (+4/-6)
src/location/providers/config.cpp (+1/-13)
src/location/providers/dummy/provider.cpp (+19/-50)
src/location/providers/dummy/provider.h (+2/-1)
src/location/providers/gps/provider.cpp (+4/-3)
src/location/providers/gps/provider.h (+4/-2)
src/location/providers/mls/provider.cpp (+11/-32)
src/location/providers/mls/provider.h (+2/-1)
src/location/providers/sirf/provider.cpp (+6/-40)
src/location/providers/sirf/provider.h (+2/-1)
src/location/providers/ubx/provider.cpp (+9/-67)
src/location/providers/ubx/provider.h (+1/-1)
src/location/util/settings.cpp (+57/-0)
src/location/util/settings.h (+64/-0)
tests/provider_registry_test.cpp (+3/-3)
To merge this branch: bzr merge lp://staging/~thomas-voss/location-service/unify-settings-backend
Reviewer Review Type Date Requested Status
Simon Fels (community) Approve
Thomas Voß Pending
Review via email: mp+323961@code.staging.launchpad.net

Commit message

Unify settings infrastructure towards provider creation.

Description of the change

Unify settings infrastructure towards provider creation.

To post a comment you must log in.
Revision history for this message
Simon Fels (morphis) wrote :

LGTM!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: