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

GCC warning fixes while compiling. #12

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

GCC warning fixes while compiling. #12

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Oct 8, 2019

I know they are only warnings but it's annoying to see a lot of text for ntohing
while compiling.

JoyHat compiling warning fix

  • "#import" is obsolete on GCC:
  • "SDL.h" is not right, changed to "SDL2/SDL.h"

Unused parameters:

  • "int argc, char *args[]" are not used is our case.
    Deleted until we got an usage to get it back.

SDL_Color missing arguments:

  • added an alpha range value: 255

SDL_rect missing arguments:

  • added a value that is not even used in the game.

JoyHat Initializing Order:

  • fixed the order for a proper initialization
    warning fixed

Missing initialization:

  • fixed the missing initialization for BoardScanner::findVerticalMatch()
    warning fixed

Unused variable fixed:

  • there's no singleton to call here
    commented code that was not used

parseInputEvent(...) waiting a return value:

  • added a return 0; to bypass the warning
    it is not like it is waiting for something

warning fixed

I know they are only warnings but it's annoying to see a lot of text for ntohing
while compiling.

JoyHat compiling warning fix
- "#import" is obsolete on GCC:
- "SDL.h" is not right, changed to "SDL2/SDL.h"

Unused parameters:
- "int argc, char *args[]" are not used is our case.
  Deleted until we got an usage to get it back.

SDL_Color missing arguments:
- added an alpha range value: 255

SDL_rect missing arguments:
- added a value that is not even used in the game.

JoyHat Initializing Order:
- fixed the order for a proper initialization
  warning fixed

Missing initialization:
- fixed the missing initialization for BoardScanner::findVerticalMatch()
  warning fixed

Unused variable fixed:
- there's no singleton to call here
  commented code that was not used

parseInputEvent(...) waiting a return value:
- added a return 0; to bypass the warning
  it is not like it is waiting for something

warning fixed
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.

0 participants