On Tue, 22 Nov 2011, Markus Armbruster wrote:
> Falls through to 32 bpp.  Harmless, because the only difference is the
> alpha component, and we're not using that.  Spotted by Coverity.
> 
> Signed-off-by: Markus Armbruster <arm...@redhat.com>

ack

>  console.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/console.c b/console.c
> index ce0429d..223f8fd 100644
> --- a/console.c
> +++ b/console.c
> @@ -1688,6 +1688,7 @@ PixelFormat qemu_default_pixelformat(int bpp)
>              pf.rbits = 8;
>              pf.gbits = 8;
>              pf.bbits = 8;
> +            break;
>          case 32:
>              pf.rmask = 0x00FF0000;
>              pf.gmask = 0x0000FF00;
> -- 
> 1.7.6.4
> 
> 

Reply via email to