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 2019/08/24 16:43:38 UTC
svn commit: r1865858 -
/pdfbox/branches/issue4569/debugger/src/main/java/org/apache/pdfbox/debugger/ui/Tree.java
Author: tilman
Date: Sat Aug 24 16:43:38 2019
New Revision: 1865858
URL: http://svn.apache.org/viewvc?rev=1865858&view=rev
Log:
PDFBOX-4071: refactor
Modified:
pdfbox/branches/issue4569/debugger/src/main/java/org/apache/pdfbox/debugger/ui/Tree.java
Modified: pdfbox/branches/issue4569/debugger/src/main/java/org/apache/pdfbox/debugger/ui/Tree.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/issue4569/debugger/src/main/java/org/apache/pdfbox/debugger/ui/Tree.java?rev=1865858&r1=1865857&r2=1865858&view=diff
==============================================================================
--- pdfbox/branches/issue4569/debugger/src/main/java/org/apache/pdfbox/debugger/ui/Tree.java (original)
+++ pdfbox/branches/issue4569/debugger/src/main/java/org/apache/pdfbox/debugger/ui/Tree.java Sat Aug 24 16:43:38 2019
@@ -104,31 +104,33 @@ public class Tree extends JTree
obj = ((ArrayEntry) obj).getValue();
}
- if (obj instanceof COSStream)
+ if (!(obj instanceof COSStream))
{
- treePopupMenu.addSeparator();
+ return;
+ }
- COSStream stream = (COSStream) obj;
- treePopupMenu.add(getStreamSaveMenu(stream, nodePath));
+ treePopupMenu.addSeparator();
- if (stream.getFilters() != null)
+ COSStream stream = (COSStream) obj;
+ treePopupMenu.add(getStreamSaveMenu(stream, nodePath));
+
+ if (stream.getFilters() != null)
+ {
+ if (stream.getFilters() instanceof COSArray && ((COSArray) stream.getFilters()).size() >= 2)
{
- if (stream.getFilters() instanceof COSArray && ((COSArray) stream.getFilters()).size() >= 2)
+ for (JMenuItem menuItem : getPartiallyDecodedStreamSaveMenu(stream))
{
- for (JMenuItem menuItem : getPartiallyDecodedStreamSaveMenu(stream))
- {
- treePopupMenu.add(menuItem);
- }
+ treePopupMenu.add(menuItem);
}
- treePopupMenu.add(getRawStreamSaveMenu(stream));
}
+ treePopupMenu.add(getRawStreamSaveMenu(stream));
+ }
- JMenuItem open = getFileOpenMenu(stream, nodePath);
- if (open != null)
- {
- treePopupMenu.addSeparator();
- treePopupMenu.add(open);
- }
+ JMenuItem open = getFileOpenMenu(stream, nodePath);
+ if (open != null)
+ {
+ treePopupMenu.addSeparator();
+ treePopupMenu.add(open);
}
}
@@ -186,23 +188,20 @@ public class Tree extends JTree
{
StringBuilder sb = new StringBuilder();
COSBase filters = cosStream.getFilters();
- if (filters != null)
+ if (filters instanceof COSName)
{
- if (filters instanceof COSName)
- {
- sb.append(((COSName) filters).getName());
- }
- else if (filters instanceof COSArray)
+ sb.append(((COSName) filters).getName());
+ }
+ else if (filters instanceof COSArray)
+ {
+ COSArray filterArray = (COSArray) filters;
+ for (int i = 0; i < filterArray.size(); i++)
{
- COSArray filterArray = (COSArray) filters;
- for (int i = 0; i < filterArray.size(); i++)
+ if (i > 0)
{
- if (i > 0)
- {
- sb.append(", ");
- }
- sb.append(((COSName) filterArray.get(i)).getName());
+ sb.append(", ");
}
+ sb.append(((COSName) filterArray.get(i)).getName());
}
}
return sb.toString();