-
Notifications
You must be signed in to change notification settings - Fork 222
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
License and thoughts #5
Comments
@livibetter @karlstav I'd suggest using MIT/X11 license. I'd agree on anything open source. (GPL, MIT, WTFPL). |
@livibetter thanks for all the input, I am verry pleased that you want to contribute! I have now licensed it under MIT. It seemed the simplest although I am a big fan of the GNU GPL as well. Coding style is a problem for me, since i don't really know any. I tend to simply ramble down the code. Maybe I should read up on that. I remember trying to use C99, but it produced a lot of other warnings and indeed an error. I think getting rid of the warnings in some other way is better. |
@karlstav sounds like you want to take on coding style, if so, good, it's better you to decide what kind of style you want for cava. I will close this issue and hold on for a while to see what you decide to do. |
I noticed a few things and I'd like to contribute, however, there is no license and I don't like to touch code without written open source license.
@karlstav would you license C.A.V.A? If yes, @anko and @CelestialWalrus do you agree on the license?
Here is a list I might do:
no consistent coding style, and issues like
\t
and space mixed in indentations and trailing whitespace, etc.This one has to be done first, or more inconsistence would be made.
makefile
, use canned recipe and turn on-Wall
and-Wextra
, etc.warnings like the following and things popping out after turning on warnings:
It's just warning, although cava works fine, but I think this should be dealt with properly.
I think this need @karlstav to decide the direction, either
-std=c99
or print out UTF8-encoded? I am not sure since I have never dealt with the Unicode, not really a C coder. (And Use ncurses instead of printing a lot? #4 related or helped?)If taking C99 route,
usleep
might need to be replaced withnanosleep
or there would be another warning.The text was updated successfully, but these errors were encountered: