Skip to content

add solution #85

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

add solution #85

wants to merge 2 commits into from

Conversation

maximumSHOT-HSE
Copy link
Collaborator

No description provided.

krinkin
krinkin reviewed May 27, 2019
View reviewed changes
Copy link
Member

@krinkin krinkin left a comment

Choose a reason for hiding this comment

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

what is the elf file?

@maximumSHOT-HSE
Copy link
Collaborator Author

It is binary file, which has been produced via gcc chat.c -o async and added to the repo by accident.

krinkin
krinkin suggested changes Jun 3, 2019
View reviewed changes
Copy link
Member

@krinkin krinkin left a comment

Choose a reason for hiding this comment

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

kkv@thinkpad:/ws/projects/os-hse-1h2019/solutions/sur/async$ gcc chat.c
chat.c:15:6: error: variably modified ‘buf1’ at file scope
char buf1[N], buf2[N];
^
chat.c:15:15: error: variably modified ‘buf2’ at file scope
char buf1[N], buf2[N];
^

@krinkin krinkin added the changes_requested Something isn't working label Jun 3, 2019
@maximumSHOT-HSE maximumSHOT-HSE added fixed and removed changes_requested Something isn't working labels Jun 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@krinkin krinkin krinkin requested changes

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
fixed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@maximumSHOT-HSE @krinkin