Re: winex11: Add a few 'fall through' comments.

2012-11-27 Thread Huw Davies
On Tue, Nov 27, 2012 at 06:44:35PM +0800, Dmitry Timoshkov wrote: > case DirectColor: > X11DRV_PALETTE_PaletteFlags |= X11DRV_PALETTE_VIRTUAL; > /* fall through */ > case GrayScale: > > would be more clear IMHO. Right now it's not obvious to which case statement > 'fall throu

Re: winex11: Add a few 'fall through' comments.

2012-11-27 Thread Dmitry Timoshkov
Huw Davies wrote: > case DirectColor: > X11DRV_PALETTE_PaletteFlags |= X11DRV_PALETTE_VIRTUAL; > -case GrayScale: > +case GrayScale: /* fall through */ case DirectColor: X11DRV_PALETTE_PaletteFlags |= X11DRV_PALETTE_VIRTUAL; /* fall through */ case Gray