[Patch]: Ncurses frame drawing

Corinna Vinschen cygwin-patches@cygwin.com
Wed Oct 15 08:23:00 GMT 2003


Good idea, that.  I've applied the patch with two small change:

On Tue, Oct 14, 2003 at 10:26:27PM +0200, Micha Nelissen wrote:
> Index: fhandler_console.cc
> ===================================================================
> RCS file: /cvs/src/src/winsup/cygwin/fhandler_console.cc,v
> retrieving revision 1.115
> diff -u -w -r1.115 fhandler_console.cc
> --- fhandler_console.cc	27 Sep 2003 02:36:50 -0000	1.115
> +++ fhandler_console.cc	14 Oct 2003 20:20:51 -0000
> @@ -66,6 +66,13 @@
>  inline BOOL
>  str_to_con (char *d, const char *s, DWORD sz)
>  {
> +  if (alternate_charset_active)
> +    {
> +      /* no translation when alternate charset is active */
> +      memcpy(d, s, sz);
> +      return TRUE;
> +    }
> +  else
>    return cp_convert (GetConsoleOutputCP (), d, get_cp (), s, sz);
  ^^^^^^^^^^
  this requires to either reindent the `return' statement or, since the
  `then' part of the if-clause returns anyway, do without the else at all.
  I removed the `else' here.

> 2003-10-13  Micha Nelissen  <M.Nelissen@student.tue.nl>
> 
>         * fhandler_console.cc (char_command): Add escape sequence for codepage
>         ansi <-> oem switching for ncurses frame drawing capabilities.
> 
>         * dcrt0.cc: Add local variable alternate_charset_active.
> 
>         * winsup.h: Add global external variable alternate_charset_active.

The ChangeLog entries are fine, I've just removed the empty lines between
them.  We're using empty lines mostly only to separate two distinct groups
of patches in one entry.

Thanks,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Developer                                mailto:cygwin@cygwin.com
Red Hat, Inc.



More information about the Cygwin-patches mailing list