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

index/numPages/numSlides: unsigned => int #197

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

vedgy
Copy link
Contributor

@vedgy vedgy commented Feb 6, 2021

This change gets rid of a few GCC's -Wsign-compare warnings and several explicit casts because of different signedness.

Make several getters const member functions rather than slots.

Rename several signals from numPages() to numPagesChanged() to simplify using function pointer-based signal-slot connection syntax.

This change gets rid of a few GCC's -Wsign-compare warnings and several
explicit casts because of different signedness.

Make several getters const member functions rather than slots.

Rename several signals from numPages() to numPagesChanged() to
simplify using function pointer-based signal-slot connection syntax.
@vedgy
Copy link
Contributor Author

vedgy commented Mar 1, 2021

I've cleaned up Render code somewhat in https://github.com/vedgy/yacreader/commits/clean-up-render based on this branch. I'll create a separate pull request after this one is merged to avoid resolving merge conflicts repeatedly.

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.

1 participant