Rename pdfview to avoid linker warnings#52
Open
carsten-wenderdel wants to merge 2 commits intomindbrix:masterfrom
Open
Rename pdfview to avoid linker warnings#52carsten-wenderdel wants to merge 2 commits intomindbrix:masterfrom
carsten-wenderdel wants to merge 2 commits intomindbrix:masterfrom
Conversation
added 2 commits
November 14, 2016 18:20
* rename-pdfview: Rename PDFView to avoid linker warnings about two classes with same name
|
I have come across this problem as well. It took me quite sometime to find out the naming of this specific file was the problem. When I opened the PDFView description in Xcode, it would display PDFKit > PDFView header description but the compiler would compile UIImage-PDF > PDFView object instead. In run time, my project would throw [PDFView setAutoScales:]: unrecognized selector sent to instance whenever I called a property or method on local PDFView variable because the called method/function did not exist in the compiled UIImage-PDF > PDFView. Renaming the PDFView resolves the naming mix-up. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Apple has an internal class also called PDFView. As it's not defined which class should be used, the linker gives a warning about that. To avoid that, PDFView has been renamed to NTBPDFView.