Author: tilman
Date: Sun Feb 23 08:40:04 2025
New Revision: 1923999
URL: http://svn.apache.org/viewvc?rev=1923999&view=rev
Log:
PDFBOX-5961: refactor
Modified:
pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cmap/CMap.java
Modified: pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cmap/CMap.java
URL:
http://svn.apache.org/viewvc/pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cmap/CMap.java?rev=1923999&r1=1923998&r2=1923999&view=diff
==============================================================================
--- pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cmap/CMap.java
(original)
+++ pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cmap/CMap.java Sun
Feb 23 08:40:04 2025
@@ -349,24 +349,24 @@ public class CMap
*/
void addCharMapping(byte[] codes, String unicode)
{
- if (codes.length == 1)
+ switch (codes.length)
{
- charToUnicodeOneByte.put(CMapStrings.getIndexValue(codes),
unicode);
- unicodeToByteCodes.put(unicode, CMapStrings.getByteValue(codes));
- }
- else if (codes.length == 2)
- {
- charToUnicodeTwoBytes.put(CMapStrings.getIndexValue(codes),
unicode);
- unicodeToByteCodes.put(unicode, CMapStrings.getByteValue(codes));
- }
- else if (codes.length == 3 || codes.length == 4)
- {
- charToUnicodeMoreBytes.put(toInt(codes), unicode);
- unicodeToByteCodes.put(unicode, codes.clone());
- }
- else
- {
- LOG.warn("Mappings with more than 4 bytes (here: {}) aren't
supported yet", codes.length);
+ case 1:
+ charToUnicodeOneByte.put(CMapStrings.getIndexValue(codes),
unicode);
+ unicodeToByteCodes.put(unicode,
CMapStrings.getByteValue(codes));
+ break;
+ case 2:
+ charToUnicodeTwoBytes.put(CMapStrings.getIndexValue(codes),
unicode);
+ unicodeToByteCodes.put(unicode,
CMapStrings.getByteValue(codes));
+ break;
+ case 3:
+ case 4:
+ charToUnicodeMoreBytes.put(toInt(codes), unicode);
+ unicodeToByteCodes.put(unicode, codes.clone());
+ break;
+ default:
+ LOG.warn("Mappings with more than 4 bytes (here: {}) aren't
supported yet", codes.length);
+ break;
}
// fixme: ugly little hack
if (SPACE.equals(unicode))
