Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename pdfview to avoid linker warnings #52

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

carsten-wenderdel
Copy link

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.

Carsten Wenderdel added 2 commits November 14, 2016 18:20
* rename-pdfview:
  Rename PDFView to avoid linker warnings about two classes with same name
@emrahu
Copy link

emrahu commented Jan 8, 2020

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants