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

Small fixes and improved README #1

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

Conversation

@zeehio
Copy link

@zeehio zeehio commented Sep 7, 2016

Hi,

Thanks for your work on this software.

These three commits provide:

  • a fix to a g++ warning in ubuntu (the change from long to size_t)
  • correct compilation under gcc-5 in Ubuntu 16.04
  • Option to disable the compilation with static libraries so the ubuntu build runs fine
  • isnan was missing in fixedpoint.h
  • The README is formatted in markdown and has detail on how to build sqct in Ubuntu.

Feel free to merge this Pull Request if you like

Sergio

zeehio added 3 commits Sep 7, 2016
This commit:
- enables c++11 support as requested in the README.
- Fixes a warning on appr/topt-bfs.h
- Fixes a compilation error due to missing isnan in fixedpoint.h
Static linking fails on Ubuntu 16.04, so I have put it under a
command line switch option. The default has not changed.
- Use markdown to provide better formatting
- Describe detailed build instructions in Ubuntu
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant