Subject: Re: Mouse console support
To: Julio Merino <>
From: Bang Jun-Young <>
List: tech-kern
Date: 02/02/2002 02:46:15
On Wed, Jan 30, 2002 at 07:58:37PM +0100, Julio Merino wrote:
> Should I send it to the list for code veritication, or only to a
> commiter?

I had a look at the OpenBSD code before, and I came to the
following conclusion: It shouldn't be committed as it is. Other
than the problem I said in the previous mail, there are too many
places in which variables of uppercase names are used in
assignment statements:

	CPY_START = row * N_COLS;

When I first saw the code, I felt like I was looking at 70's BASIC
source. IMHO, in the tradition of C only constants or variables of
which values are not modified should be written in uppercases (note
I'm not saying macro functions). They are defined in wsmoused.h as

#define CPY_START	(sc->sc_focus->cpy_start)
#define MOUSE_FLAGS 	(sc->sc_focus->mouse_flags)

It would be good if those names were converted to lowercase ones and
prefixed with wsm_ or something like that. 

Also, it should be cleanly separated from the rest of sources so
that it is possible to compile optionally, i.e. options WSCONS_WSMOUSED
in the kernel config file, if considering there are quite a number
of people who issue objection to any new features that might make their
kernels bigger, even if it's relatively small and quite useful (most of
them are *real* i386 users, I guess).


Bang Jun-Young <>