summaryrefslogtreecommitdiff
path: root/platform/java/src
diff options
context:
space:
mode:
authorRobin Watts <Robin.Watts@artifex.com>2017-06-03 21:27:33 -0700
committerRobin Watts <Robin.Watts@artifex.com>2017-06-03 21:27:33 -0700
commit7f0a9b9f4448f7f2e87313e9e147b631b687e81b (patch)
tree5259a54b29e55d19fe65d75ac35cbb56022e591c /platform/java/src
parent876f1fe8aaa9b110b3211fedbe5decbd67cf8c45 (diff)
downloadmupdf-7f0a9b9f4448f7f2e87313e9e147b631b687e81b.tar.xz
Tweak pdf_graft_map API.
Passing a pdf_document to pdf_graft_object to specify the source document is redundant, as if we need to know the document, it will be pickled into the object we are copying. Similarly, repeatedly having to pass the destination document seems silly when we can just pickle it into the map too (and this removes the possibility of people using a different destination document part way through). This leaves to simplifying the pdf_graft_object call, at the expense of splitting it into 2 calls - one with a map, and one without. Also, we can delay the creation of the mapping table until we are first asked to copy an object that requires deep copying. This avoids us ever having to manually pass in the source document. This has knock-on effects in the java and javascript classes, but with the advantage of being clearer in the end (IMHO). Conflicts: include/mupdf/pdf/document.h
Diffstat (limited to 'platform/java/src')
-rw-r--r--platform/java/src/com/artifex/mupdf/fitz/PDFDocument.java2
-rw-r--r--platform/java/src/com/artifex/mupdf/fitz/PDFGraftMap.java2
2 files changed, 3 insertions, 1 deletions
diff --git a/platform/java/src/com/artifex/mupdf/fitz/PDFDocument.java b/platform/java/src/com/artifex/mupdf/fitz/PDFDocument.java
index a925ac90..5c0cc5ec 100644
--- a/platform/java/src/com/artifex/mupdf/fitz/PDFDocument.java
+++ b/platform/java/src/com/artifex/mupdf/fitz/PDFDocument.java
@@ -40,7 +40,7 @@ public class PDFDocument extends Document
}
public native PDFGraftMap newPDFGraftMap();
- public native PDFObject graftObject(PDFDocument src, PDFObject obj, PDFGraftMap map);
+ public native PDFObject graftObject(PDFObject obj);
private native PDFObject addStreamBuffer(Buffer buf, Object obj, boolean compressed);
private native PDFObject addStreamString(String str, Object obj, boolean compressed);
diff --git a/platform/java/src/com/artifex/mupdf/fitz/PDFGraftMap.java b/platform/java/src/com/artifex/mupdf/fitz/PDFGraftMap.java
index 5587ed2c..fedace4f 100644
--- a/platform/java/src/com/artifex/mupdf/fitz/PDFGraftMap.java
+++ b/platform/java/src/com/artifex/mupdf/fitz/PDFGraftMap.java
@@ -11,6 +11,8 @@ public class PDFGraftMap
pointer = 0;
}
+ public native PDFObject graftObject(PDFObject obj);
+
private PDFGraftMap(long p) {
pointer = p;
}