| author | zecke <zecke> | 2004-05-21 12:54:17 (UTC) |
|---|---|---|
| committer | zecke <zecke> | 2004-05-21 12:54:17 (UTC) |
| commit | 4d1d8e2c39b764f07dc4af5da5bf142b2c5436b4 (patch) (side-by-side diff) | |
| tree | b1ad90ebab404524f50e37a3a468a26ee8240a16 /libopie2 | |
| parent | 1859469bb409e665dff20994923887f132f9790f (diff) | |
| download | opie-4d1d8e2c39b764f07dc4af5da5bf142b2c5436b4.zip opie-4d1d8e2c39b764f07dc4af5da5bf142b2c5436b4.tar.gz opie-4d1d8e2c39b764f07dc4af5da5bf142b2c5436b4.tar.bz2 | |
Rule of Thumb: If it is 3:00am and you've suddenly something that compiles it is a
step in the right direction but wrong...
Now the question is where to get the brown paperbacks again?! Maybe we get a discount...
Anyway:
adding a break in switch/case is most likely a good idea. Before we only used return
| -rw-r--r-- | libopie2/opiepim/core/opimaccessfactory.h | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/libopie2/opiepim/core/opimaccessfactory.h b/libopie2/opiepim/core/opimaccessfactory.h index a80e67c..ee55ba5 100644 --- a/libopie2/opiepim/core/opimaccessfactory.h +++ b/libopie2/opiepim/core/opimaccessfactory.h @@ -81,12 +81,15 @@ class OPimAccessFactory OPimBase *base; switch ( type ){ case OPimGlobal::TODOLIST: base = new OPimTodoAccess( OBackendFactory<OPimTodoAccessBackend>::create( type, dbStyle, appName ) ); + break; case OPimGlobal::CONTACTLIST: base = new OPimContactAccess( QString::null, QString::null, OBackendFactory<OPimContactAccessBackend>::create( type, dbStyle, appName ) ); + break; case OPimGlobal::DATEBOOK: base = new ODateBookAccess( OBackendFactory<ODateBookAccessBackend>::create( type, dbStyle, appName ) ); + break; default: return 0l; } return static_cast<T*>( base ); |