From: "andrzej zaborowski" <balrogg@gmail.com>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] ncurses: resize console if required
Date: Mon, 28 Apr 2008 21:17:56 +0200 [thread overview]
Message-ID: <fb249edb0804281217w6fbf099ds7b84f79f4764afce@mail.gmail.com> (raw)
In-Reply-To: <20080428173113.GB8787@tapir>
On 28/04/2008, Carlo Marcelo Arenas Belon <carenas@sajinet.com.pe> wrote:
> On Mon, Apr 28, 2008 at 11:14:15AM +0100, Thiemo Seufer wrote:
> > Carlo Marcelo Arenas Belon wrote:
> > > The following patch instructs qemu to print an escape command to resize the
> > > curses console to 80x25 if detected to have a different geometry (xterm and
> > > friends use 80x24 by default).
> >
> > Sounds like this should be specific to the PC/VGA emulation.
>
>
> not sure what you mean, but the curses.c emulation assumes (and is hardcoded)
> to a 80x25 console anyway.
It's only this way because SDL uses 640x400, but it can change when
the vga starts living. For example the text mode ms windows installer
(80x43 or something) worked fine for me with -curses.
>
> this patch only checks the window size of the console that is being used and
> sends it an escape command to change its size to what the curses console will
> use so that they match; otherwise if starting qemu from an 80x24 xterm with
> -curses you won't be able to see the last line of the console until the window
> is resized.
To be honest I wasn't aware that there's a sequence to request
terminal size change. "man console_codes" on my system doesn't list
this sequence and also it doesn't seem to have any effect in
gnome-terminal. This backend however is the curses backend, not
terminal backend and even if the sequence worked, we should only use
what curses can provide (this means that we don't even need a unix
terminal). It seems that curses generally wants programs to be aware
of the terminal size and conform, rather than have any say. So that
would be a policy change also.
Regards
--
Please do not print this email unless absolutely necessary. Spread
environmental awareness.
next prev parent reply other threads:[~2008-04-28 19:18 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-04-28 3:54 [Qemu-devel] [PATCH] ncurses: resize console if required Carlo Marcelo Arenas Belon
2008-04-28 8:58 ` Samuel Thibault
2008-04-28 17:56 ` Carlo Marcelo Arenas Belon
2008-04-28 23:16 ` Samuel Thibault
2008-04-28 10:14 ` Thiemo Seufer
2008-04-28 17:31 ` Carlo Marcelo Arenas Belon
2008-04-28 17:20 ` Samuel Thibault
2008-04-28 19:17 ` andrzej zaborowski [this message]
2008-04-28 23:18 ` Samuel Thibault
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=fb249edb0804281217w6fbf099ds7b84f79f4764afce@mail.gmail.com \
--to=balrogg@gmail.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).