Skip to content

Scanf has been replaced by getline - #101

Open
jridky wants to merge 2 commits into
develfrom
getline
Open

Scanf has been replaced by getline#101
jridky wants to merge 2 commits into
develfrom
getline

Conversation

@jridky

@jridky jridky commented Mar 31, 2016

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread src/client/client.c
int question(char* question, int possibilities) {
int status = NO, repeat = 1;
char answer;
char *answer = NULL, letter;

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems to me there is a memory leak because answer is never freed.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. Thank you.

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.

2 participants