Re: Question about dubious code for terminal frames

[martin rudalics <rudalics@gmx.at>]

 > I think I have the above here. It reads
 >
 >    /* Assign new sizes.  */
 >    FRAME_COLS (f) = new_text_cols;
 >    FRAME_LINES (f) = new_text_lines;

It might cause something weird in resize_frame_windows, but that's maybe
only here.

 > Anyway. My problem is setting FrameCols/Rows in adjust_frame_size. It's
 > not guaranteed that the terminal's size has indeed changed when that is
 > done.

You mean the

	FrameRows (FRAME_TTY (f)) =
	FrameCols (FRAME_TTY (f)) =

assignments?  These are from the pre-2014 change_frame_size_1 code with
the comments mostly preserved and governed by similar conditions.  Do
you think something changed here?

But I agree that for child frames with their own smaller dimensions
these assignments should be obviously skipped.

martin