Re: [PATCH] mscms: Handle bitmap formats BM_xRGBQUADS in from_bmformat( BMFORMAT format ) which is used by TranslateBitmapBits

[Hans Leidekker]

On Wed, 2010-10-20 at 12:45 +0200, Tomasz Michno wrote:
> C_SRCS = \
> -
>   profile.c
> -
>   profile.c \
> -
>   transform.c

You are copying quite a bit a infrastructure from profile.c, so it would
be better to add the tests to that file. We can always split them up
later.
> +#include "mscms_priv.h"

This is not needed.
> +void test_TranslateBitmapBits(void){
> - BOOL ret=TRUE;
> - COLOR bitmapBits[bitmapSize];
> - COLOR bitmapBitsResult[bitmapSize];
> - #ifdef HAVE_LCMS

You can't use that define in tests.
> - HPROFILE hProfile;
>
> ...
> - transform = pCreateMultiProfileTransform(&hProfile, 2, &intent, 1,
> - // test already handled BMFORMATs:

This won't work, you say you have 2 profiles but pass only one. Don't
use // for comments.