| Re: [AD] TODO List for Allegro |
[ Thread Index |
Date Index
| More lists.liballeg.org/allegro-developers Archives
]
On Mon, 2004-07-19 at 23:25 +0200, Evert Glebbeek wrote:
> On Monday 19 July 2004 22:53, Elias Pschernig wrote:
> > Does someone know why the Allegro keyboard stops working though? I think
> > we should apply the patch, but still put investigation of the problem to
> > the TODO - an ideal solution would still use Allegro's keyboard and make
> > it working.
>
> There are a few (minor) formatting problems with the patch as is, most
> noticably evil carriage-returns in the line endings.
> One thing I did notice that I wanted to ask about is this:
>
> -// Windows resource header file for the dxwindow example
>
> Allegro's source shouldn't contain C++ style comments, how about tests and
> example programs? I know this is not introduced with this patch, I'm just
> wondering if those should not all be changed to /* ... */ style comments
> either.
>
Yes, I agree, we should keep the Allegro style everywhere. It's probably
not worth though specifically looking for style violations in the
existing code and fix them. I'm myself trying to fix them whenever I
encounter them though when touching some code.
--
Elias Pschernig