You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ti...@apache.org on 2020/05/08 09:16:48 UTC
svn commit: r1877504 -
/pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/xml/PdfaExtensionHelper.java
Author: tilman
Date: Fri May 8 09:16:48 2020
New Revision: 1877504
URL: http://svn.apache.org/viewvc?rev=1877504&view=rev
Log:
PDFBOX-4071: use switch over strings
Modified:
pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/xml/PdfaExtensionHelper.java
Modified: pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/xml/PdfaExtensionHelper.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/xml/PdfaExtensionHelper.java?rev=1877504&r1=1877503&r2=1877504&view=diff
==============================================================================
--- pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/xml/PdfaExtensionHelper.java (original)
+++ pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/xml/PdfaExtensionHelper.java Fri May 8 09:16:48 2020
@@ -273,21 +273,19 @@ public final class PdfaExtensionHelper
if (pos > 0)
{
String scard = valueType.substring(0, pos);
- if ("seq".equals(scard))
+ switch (scard)
{
- card = Cardinality.Seq;
- }
- else if ("bag".equals(scard))
- {
- card = Cardinality.Bag;
- }
- else if ("alt".equals(scard))
- {
- card = Cardinality.Alt;
- }
- else
- {
- return null;
+ case "seq":
+ card = Cardinality.Seq;
+ break;
+ case "bag":
+ card = Cardinality.Bag;
+ break;
+ case "alt":
+ card = Cardinality.Alt;
+ break;
+ default:
+ return null;
}
}
String vt = valueType.substring(pos + 1);