[PATCH] coverity#705135 Missing break in switch

Julien Nabet (via Code Review) gerrit at gerrit.libreoffice.org
Sat Mar 16 23:21:34 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/2779

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/79/2779/1

coverity#705135 Missing break in switch

Change-Id: Ie610b3b10b5d83b772ad96e455d59360368086a4
---
M connectivity/source/drivers/dbase/DTable.cxx
1 file changed, 1 insertion(+), 0 deletions(-)



diff --git a/connectivity/source/drivers/dbase/DTable.cxx b/connectivity/source/drivers/dbase/DTable.cxx
index 52f5195..bb46006 100644
--- a/connectivity/source/drivers/dbase/DTable.cxx
+++ b/connectivity/source/drivers/dbase/DTable.cxx
@@ -354,6 +354,7 @@
                 break;
             case 'F':
                 aTypeName = "DECIMAL";
+                break;
             case 'N':
                 if ( aDBFColumn.db_typ == 'N' )
                     aTypeName = "NUMERIC";

-- 
To view, visit https://gerrit.libreoffice.org/2779
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie610b3b10b5d83b772ad96e455d59360368086a4
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Julien Nabet <serval2412 at yahoo.fr>



More information about the LibreOffice mailing list