Author: nick
Date: Thu Feb 20 23:20:19 2014
New Revision: 1570388

URL: http://svn.apache.org/r1570388
Log:
More helpful visio v5 exception, see bug #56171

Modified:
    
poi/trunk/src/scratchpad/src/org/apache/poi/hdgf/pointers/PointerFactory.java
    
poi/trunk/src/scratchpad/testcases/org/apache/poi/hdgf/pointers/TestPointerFactory.java

Modified: 
poi/trunk/src/scratchpad/src/org/apache/poi/hdgf/pointers/PointerFactory.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hdgf/pointers/PointerFactory.java?rev=1570388&r1=1570387&r2=1570388&view=diff
==============================================================================
--- 
poi/trunk/src/scratchpad/src/org/apache/poi/hdgf/pointers/PointerFactory.java 
(original)
+++ 
poi/trunk/src/scratchpad/src/org/apache/poi/hdgf/pointers/PointerFactory.java 
Thu Feb 20 23:20:19 2014
@@ -42,7 +42,7 @@ public final class PointerFactory {
 
                        return p;
                } else if(version == 5) {
-                       throw new RuntimeException("TODO");
+                       throw new RuntimeException("TODO Support v5 Pointers");
                } else {
                        throw new IllegalArgumentException("Visio files with 
versions below 5 are not supported, yours was " + version);
                }

Modified: 
poi/trunk/src/scratchpad/testcases/org/apache/poi/hdgf/pointers/TestPointerFactory.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/testcases/org/apache/poi/hdgf/pointers/TestPointerFactory.java?rev=1570388&r1=1570387&r2=1570388&view=diff
==============================================================================
--- 
poi/trunk/src/scratchpad/testcases/org/apache/poi/hdgf/pointers/TestPointerFactory.java
 (original)
+++ 
poi/trunk/src/scratchpad/testcases/org/apache/poi/hdgf/pointers/TestPointerFactory.java
 Thu Feb 20 23:20:19 2014
@@ -57,7 +57,7 @@ public final class TestPointerFactory ex
                        fail();
                } catch(RuntimeException e) {
                        // Still to do
-                       assertEquals("TODO", e.getMessage());
+                       assertEquals("TODO Support v5 Pointers", 
e.getMessage());
                }
        }
 



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to