-
Notifications
You must be signed in to change notification settings - Fork 291
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
Update license headers and remove redundant file name comment. #431
Conversation
Reviewed 90 of 90 files at r1. Comments from Reviewable |
b9f03a0
to
44c36dc
Compare
Reviewed 90 of 90 files at r1. toxcore/util.c, line 5 at r1 (raw file):
You didn't remove this person's name from any other files. Is this intentional? Comments from Reviewable |
"All rights reserved" was incorrect. The project was licensed under GPL3, which means a lot of rights are licensed to everybody in the world, i.e. not reserved to the "Tox Project".
44c36dc
to
8f96ca6
Compare
Review status: 89 of 90 files reviewed at latest revision, 1 unresolved discussion. toxcore/util.c, line 5 at r1 (raw file): Previously, robinlinden (Robin Lindén) wrote…
No, added back. Comments from Reviewable |
Reviewed 1 of 1 files at r2. Comments from Reviewable |
testing/av_test.c, line 22 at r2 (raw file):
Was this intentionally removed? Comments from Reviewable |
toxcore/crypto_core_mem.c, line 21 at r2 (raw file):
Unrelated change. Comments from Reviewable |
toxav/ring_buffer.c, line 4 at r2 (raw file):
What pluto has to do with toxav? Comments from Reviewable |
toxcore/crypto_core_mem.c, line 21 at r2 (raw file): Previously, nurupo wrote…
There are actually quite of few like this. Comments from Reviewable |
Is the unicode copyright symbol somehow better than Comments from Reviewable |
"All rights reserved" was incorrect. The project was licensed under GPL3,
which means a lot of rights are licensed to everybody in the world, i.e.
not reserved to the "Tox Project".
This change is