Use explicit constants for smallest size of display rows and columns.

This commit is contained in:
Renaud 2021-08-31 12:55:10 +08:00
parent f96ace997f
commit 64eb0d5deb
3 changed files with 26 additions and 20 deletions

View File

@ -136,9 +136,13 @@ static void vtalloc( int maxrow, int maxcol) {
}
void updmargin( void) {
#define MINMARGIN 3 /* MINMARGIN - 1 enough for $ + prev before current */
#if MINCOLS < 2 * MINMARGIN + 1
# error "MINCOLS and MINMARGIN are not consistent"
#endif
term.t_margin = term.t_ncol / 10 ;
if( term.t_margin < 3) /* t_margin - 1 enough for $ + prev before current */
term.t_margin = 3 ;
if( term.t_margin < MINMARGIN)
term.t_margin = MINMARGIN ;
term.t_scrsiz = term.t_ncol - 2 * term.t_margin ;
}
@ -1327,11 +1331,11 @@ static void sizesignal( int signr) {
static void newscreensize( int h, int w) {
chg_width = chg_height = 0 ;
vtfree() ;
if( h < 3)
h = 3 ;
if( h < MINROWS)
h = MINROWS ;
if( w < 10)
w = 10 ;
if( w < MINCOLS)
w = MINCOLS ;
vtalloc( h, w) ;
if( h <= term.t_mrow)

View File

@ -8,6 +8,9 @@
#include "names.h" /* BINDABLE() */
#include "utf8.h" /* unicode_t */
#define MINROWS 3
#define MINCOLS 10
extern int mpresf ; /* Stuff in message line */
extern int scrollcount ; /* number of lines to scroll */
extern int discmd ; /* display command flag */

View File

@ -561,6 +561,11 @@ BINDABLE( restwnd) {
}
static void adjust( window_p wp, int screenrows) {
wp->w_ntrows = screenrows - wp->w_toprow - 2 ;
wp->w_flag |= WFHARD | WFMODE ;
}
/* resize the screen, re-writing the screen
*
* int f; default flag
@ -574,7 +579,7 @@ BBINDABLE( newsize) {
n = term.t_mrow ;
/* make sure it's in range */
if( n < 3 || n > term.t_mrow)
if( n < MINROWS || n > term.t_mrow)
return mloutfail( "%%Screen size out of range") ;
if( term.t_nrow == n - 1)
@ -587,8 +592,7 @@ BBINDABLE( newsize) {
;
/* and enlarge it as needed */
wp->w_ntrows = n - wp->w_toprow - 2 ;
wp->w_flag |= WFHARD | WFMODE ;
adjust( wp, n) ;
} else {
/* new size is smaller */
/* rebuild the window structure */
@ -598,10 +602,9 @@ BBINDABLE( newsize) {
wp = nextwp ;
nextwp = wp->w_wndp ;
if( wp->w_toprow == n - 2) {
lastwp->w_ntrows = n - lastwp->w_toprow - 2 ;
lastwp->w_flag |= WFHARD | WFMODE ;
}
/* expand previous window if current would have zero lines */
if( wp->w_toprow == n - 2)
adjust( lastwp, n) ;
/* get rid of it if it is too low */
if( wp->w_toprow >= n - 2) {
@ -624,12 +627,8 @@ BBINDABLE( newsize) {
lastwp->w_wndp = NULL ;
} else {
/* need to change this window size? */
int lastline = wp->w_toprow + wp->w_ntrows - 1 ;
if( lastline >= n - 2) {
wp->w_ntrows = n - wp->w_toprow - 2 ;
assert( wp->w_ntrows) ;
wp->w_flag |= WFHARD | WFMODE ;
}
if( (wp->w_toprow + wp->w_ntrows - 1) >= n - 2)
adjust( wp, n) ;
lastwp = wp ;
}
@ -654,7 +653,7 @@ BBINDABLE( newwidth) {
n = term.t_mcol ;
/* make sure it's in range */
if( n < 10 || n > term.t_mcol)
if( n < MINCOLS || n > term.t_mcol)
return mloutfail( "%%Screen width out of range") ;
/* otherwise, just re-width it (no big deal) */