From 4d1d8e2c39b764f07dc4af5da5bf142b2c5436b4 Mon Sep 17 00:00:00 2001
From: zecke <zecke>
Date: Fri, 21 May 2004 12:54:17 +0000
Subject: 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
---
(limited to 'libopie2/opiepim')

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
@@ -82,10 +82,13 @@ class OPimAccessFactory
 		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;
 		}
--
cgit v0.9.0.2