add break statement, although differences in getLongBinaryValue and getLongStringValu...

git-svn-id: https://svn.code.sf.net/p/jackcess/code/jackcess/trunk@49 f203690c-595d-4dc9-a70b-905162fa7fd2

diff --git a/src/java/com/healthmarketscience/jackcess/Column.java b/src/java/com/healthmarketscience/jackcess/Column.java
index 822fb2bb31671c378df8978284b41e5769052e58..aa0b18eb042a259f33e21bdd3677bb904ba14325 100644 (file)
--- a/src/java/com/healthmarketscience/jackcess/Column.java
+++ b/src/java/com/healthmarketscience/jackcess/Column.java
@@ -340,7 +340,6 @@ public class Column implements Comparable<Column> {
    * @param lvalDefinition Column value that points to an LVAL record
    * @return The LVAL data
    */
-  @SuppressWarnings("fallthrough")
   private byte[] getLongBinaryValue(byte[] lvalDefinition) throws IOException {
     ByteBuffer def = ByteBuffer.wrap(lvalDefinition);
     def.order(ByteOrder.LITTLE_ENDIAN);
@@ -368,8 +367,10 @@ public class Column implements Comparable<Column> {
         lvalPage.get(rtn);
         break;
       case LONG_VALUE_TYPE_THIS_PAGE:
+        // FIXME, is this correct?
         def.getLong();  //Skip over lval_dp and unknown
         def.get(rtn);
+        break;
       case LONG_VALUE_TYPE_OTHER_PAGES:
         //XXX
         return null;
@@ -383,7 +384,6 @@ public class Column implements Comparable<Column> {
    * @param lvalDefinition Column value that points to an LVAL record
    * @return The LVAL data
    */
-  @SuppressWarnings("fallthrough")
   private String getLongStringValue(byte[] lvalDefinition) throws IOException {
     ByteBuffer def = ByteBuffer.wrap(lvalDefinition);
     def.order(ByteOrder.LITTLE_ENDIAN);