Skip to content

Conversation

@asteriosos
Copy link

gcc compile error fixed.

gcc -DHAVE_CONFIG_H -I. -g -O2 -MT cmatrix.o -MD -MP -MF .deps/cmatrix.Tpo -c -o cmatrix.o cmatrix.c
cmatrix.c: In function ‘resize_screen’:
cmatrix.c:277:10: error: lvalue required as left operand of assignment
COLS = win.ws_col;
^
cmatrix.c:278:11: error: lvalue required as left operand of assignment
LINES = win.ws_row;
^
cmatrix.c:282:15: error: lvalue required as left operand of assignment
LINES = 10;
^
cmatrix.c:285:14: error: lvalue required as left operand of assignment
COLS = 10;

gcc compile error fixed.

gcc -DHAVE_CONFIG_H -I.     -g -O2 -MT cmatrix.o -MD -MP -MF .deps/cmatrix.Tpo -c -o cmatrix.o cmatrix.c
cmatrix.c: In function ‘resize_screen’:
cmatrix.c:277:10: error: lvalue required as left operand of assignment
     COLS = win.ws_col;
          ^
cmatrix.c:278:11: error: lvalue required as left operand of assignment
     LINES = win.ws_row;
           ^
cmatrix.c:282:15: error: lvalue required as left operand of assignment
         LINES = 10;
               ^
cmatrix.c:285:14: error: lvalue required as left operand of assignment
         COLS = 10;
@space-pagan
Copy link

Changing an assignment to a comparison isn't much of a fix, in my opinion. You're just avoiding the issue altogether.

Copy link
Owner

@abishekvashok abishekvashok left a comment

Choose a reason for hiding this comment

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

Heyya, I tried this patch and its not resizing on my machine. Can you pls check on it. There seems to be a suggestion by @space-pagan as well ~ maybe give it a go!

@abishekvashok abishekvashok added the Changes requested Changes have been requested for this PR. label Feb 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changes requested Changes have been requested for this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants