lsm/PdfPig
1
0
Fork 0
mirror of https://github.com/UglyToad/PdfPig.git synced 2025-09-19 02:37:56 +08:00
Code Issues Actions 5 Packages Projects Releases Wiki Activity

Removed the possibility of deepCopy a token.

Browse Source 
Anyway, the DataToken properties most are readonly, If you want to change something. You have to crate a new token anyway.  Discussion: https://github.com/UglyToad/PdfPig/pull/150
This commit is contained in:
InusualZ
2020-03-14 09:53:42 -04:00
committed by Eliot Jones
parent 3abe210c6f
commit bd6b03c2e8
1 changed files with 9 additions and 41 deletions
Download Patch File Download Diff File Expand all files Collapse all files

50
src/UglyToad.PdfPig/Writer/PdfMerger.cs
View File

@@ -274,9 +274,8 @@
/// </summary>
/// <param name="tokenToCopy">Token to inspect for reference</param>
/// <param name="tokenScanner">scanner get the content from the original document</param>
/// <param name="deepCopy">Weather we want to create a new token for basic token(Ex. NumericToken) because the original could be modified</param>
/// <returns>A reference of the token that was copied. With all the reference updated</returns>
private IToken CopyToken(IToken tokenToCopy, IPdfTokenScanner tokenScanner, bool deepCopy = false)
private IToken CopyToken(IToken tokenToCopy, IPdfTokenScanner tokenScanner)
{
// This token need to be deep copied, because they could contain reference. So we have to update them.
switch (tokenToCopy)
@@ -288,7 +287,7 @@
{
var name = setPair.Key;
var token = setPair.Value;
newContent.Add(NameToken.Create(name), CopyToken(token, tokenScanner, deepCopy));
newContent.Add(NameToken.Create(name), CopyToken(token, tokenScanner));
}
return new DictionaryToken(newContent);
@@ -298,7 +297,7 @@
var newArray = new List<IToken>(arrayToken.Length);
foreach (var token in arrayToken.Data)
{
newArray.Add(CopyToken(token, tokenScanner, deepCopy));
newArray.Add(CopyToken(token, tokenScanner));
}
return new ArrayToken(newArray);
@@ -314,7 +313,7 @@
Debug.Assert(!(tokenObject is IndirectReferenceToken));
var newToken = CopyToken(tokenObject, tokenScanner, deepCopy);
var newToken = CopyToken(tokenObject, tokenScanner);
newReferenceToken = context.WriteToken(newToken);
referencesFromDocument.Add(referenceToken, newReferenceToken);
@@ -323,53 +322,22 @@
}
case StreamToken streamToken:
{
var properties = CopyToken(streamToken.StreamDictionary, tokenScanner, deepCopy) as DictionaryToken;
var properties = CopyToken(streamToken.StreamDictionary, tokenScanner) as DictionaryToken;
Debug.Assert(properties != null);
var bytes = deepCopy ? new List<byte>(streamToken.Data) : streamToken.Data;
var bytes = streamToken.Data;
return new StreamToken(properties, bytes);
}
case ObjectToken _:
{
// This is because, since we don't write token directly to the stream. So can't know the offset.
// The token would be invalid. Although I don't think the copy of an object token would ever happen
// Since we don't write token directly to the stream.
// We can't know the offset. Therefore the token would be invalid
throw new NotSupportedException("Copying a Object token is not supported");
}
}
if (!deepCopy)
return tokenToCopy;
// This tokens can't be deep copied
if (tokenToCopy is EndOfLineToken || tokenToCopy is NullToken || tokenToCopy is BooleanToken)
return tokenToCopy;
switch (tokenToCopy)
{
case CommentToken commentToken:
return new CommentToken(commentToken.Data);
case HexToken hexToken:
return new HexToken(hexToken.Data.ToCharArray());
case InlineImageDataToken imageDataToken:
return new InlineImageDataToken(imageDataToken.Data);
case NameToken nameToken:
return NameToken.Create(nameToken.Data);
case NumericToken numericToken:
return new NumericToken(numericToken.Data);
case OperatorToken operatorToken:
return OperatorToken.Create(operatorToken.Data);
case StringToken stringToken:
return new StringToken(stringToken.Data, stringToken.EncodedWith);
}
throw new NotSupportedException($"Tried to deep copy {tokenToCopy.GetType()}");
return tokenToCopy;
}
private static bool IgnoreKeyForPagesNode(KeyValuePair<string, IToken> token)