Skip to content
This repository has been archived by the owner on Mar 28, 2023. It is now read-only.

Fix format string, include and other minor stuff #1

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

Conversation

gsauthof
Copy link

@gsauthof gsauthof commented Sep 7, 2019

This pull request fixes a format string, replaces the forward declaration with the right include and fixes the indentation.

@CLAassistant
Copy link

CLAassistant commented Sep 7, 2019

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@gsauthof
Copy link
Author

I don't think that this contribution achieves the 'threshold of originality' (when it comes to copyright).

I'm not a lawyer and I really don't have time to read your fine Contributor License Agreement.

Thus, I'm not going to sign it.

If you really think that this is obstacle because you assume that the threshold of originality is reached I'm willing to license my contribution under a very permissive license like the MIT-0 (which doesn't even require attribution) and/or release it into the public domain.

@artemdinaburg
Copy link
Collaborator

I just saw this was here :). The CLA bot is on for everything so I can't merge your contribution directly, but this is GPLv2 anyway so I will gladly incorporate the small fixes as a separate commit.

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

Successfully merging this pull request may close these issues.

3 participants