diff options
-rw-r--r-- | gdk-pixbuf/ChangeLog | 5 | ||||
-rw-r--r-- | gdk-pixbuf/io-bmp.c | 347 |
2 files changed, 284 insertions, 68 deletions
diff --git a/gdk-pixbuf/ChangeLog b/gdk-pixbuf/ChangeLog index 3c7258bc47..940dcdc13b 100644 --- a/gdk-pixbuf/ChangeLog +++ b/gdk-pixbuf/ChangeLog @@ -1,3 +1,8 @@ +1999-11-24 Jonathan Blandford <jrb@redhat.com> + + * src/io-bmp.c (OneLine24): new patch from Arjan to handle + compressed bmps. + 1999-11-24 James Henstridge <james@daa.com.au> * configure.in (AC_OUTPUT): altered hack to place the symlink in the diff --git a/gdk-pixbuf/io-bmp.c b/gdk-pixbuf/io-bmp.c index d40ce7820b..dc94b40dca 100644 --- a/gdk-pixbuf/io-bmp.c +++ b/gdk-pixbuf/io-bmp.c @@ -47,29 +47,70 @@ struct headerpair { guint width; guint height; guint depth; + guint Negative; /* Negative = 1 -> top down BMP, + Negative = 0 -> bottom up BMP */ }; +/* + +These structures are actually dummies. These are according to +the "Windows API reference guide volume II" as written by Borland, +but GCC fiddles with the alignment of the internal members. + +*/ + struct BitmapFileHeader { - gushort bfType; - guint bfSize; - guint reserverd; - guint bfOffbits; + gushort bfType; + guint bfSize; + guint reserverd; + guint bfOffbits; }; struct BitmapInfoHeader { - guint biSize; - guint biWidth; - guint biHeight; - gushort biPlanes; - gushort biBitCount; - guint biCompression; - guint biSizeImage; - guint biXPelsPerMeter; - guint biYPelsPerMeter; - guint biClrUsed; - guint biClrImportant; + guint biSize; + guint biWidth; + guint biHeight; + gushort biPlanes; + gushort biBitCount; + guint biCompression; + guint biSizeImage; + guint biXPelsPerMeter; + guint biYPelsPerMeter; + guint biClrUsed; + guint biClrImportant; }; +static void DumpBIH(unsigned char *BIH) +{ /* For debugging */ + printf("biSize = %i \n", + (BIH[3] << 24) + (BIH[2] << 16) + (BIH[1] << 8) + (BIH[0])); + printf("biWidth = %i \n", + (BIH[7] << 24) + (BIH[6] << 16) + (BIH[5] << 8) + (BIH[4])); + printf("biHeight = %i \n", + (BIH[11] << 24) + (BIH[10] << 16) + (BIH[9] << 8) + + (BIH[8])); + printf("biPlanes = %i \n", (BIH[13] << 8) + (BIH[12])); + printf("biBitCount = %i \n", (BIH[15] << 8) + (BIH[14])); + printf("biCompress = %i \n", + (BIH[19] << 24) + (BIH[18] << 16) + (BIH[17] << 8) + + (BIH[16])); + printf("biSizeImage = %i \n", + (BIH[23] << 24) + (BIH[22] << 16) + (BIH[21] << 8) + + (BIH[20])); + printf("biXPels = %i \n", + (BIH[27] << 24) + (BIH[26] << 16) + (BIH[25] << 8) + + (BIH[24])); + printf("biYPels = %i \n", + (BIH[31] << 24) + (BIH[30] << 16) + (BIH[29] << 8) + + (BIH[28])); + printf("biClrUsed = %i \n", + (BIH[35] << 24) + (BIH[34] << 16) + (BIH[33] << 8) + + (BIH[32])); + printf("biClrImprtnt= %i \n", + (BIH[39] << 24) + (BIH[38] << 16) + (BIH[37] << 8) + + (BIH[36])); +} + /* This does a byte-order swap. Does glib have something like be32_to_cpu() ?? @@ -91,7 +132,20 @@ static void free_buffer(gpointer user_data, gpointer data) } - +/* Data needed for the "state" during decompression */ +struct bmp_compression_state { + gint phase; /* 0 = clean, + 1 = count received + 2 = escape received + 3 = in "raw" run + 4 = Relative part 1 is next + 5 = Relative part 2 is next + 6 = end of image -> No more input allowed + */ + gint RunCount; + gint XDelta; + gint YDelta; +}; /* Progressive loading */ @@ -114,13 +168,11 @@ struct bmp_progressive_state { 8 = 8 bit colormapped 1 = 1 bit bitonal */ + gint Compressed; + struct bmp_compression_state compr; struct headerpair Header; /* Decoded (BE->CPU) header */ - - struct BitmapFileHeader BFH; - struct BitmapInfoHeader BIH; - GdkPixbuf *pixbuf; /* Our "target" */ @@ -141,19 +193,21 @@ GdkPixbuf *image_load(FILE * f) size_t length; struct bmp_progressive_state *State; int fd; - + GdkPixbuf *pb; - + State = image_begin_load(NULL, NULL, NULL); - membuf = g_malloc(4096); - + g_assert(membuf != NULL); - + + while (feof(f) == 0) { length = fread(membuf, 1, 4096, f); - image_load_increment(State, membuf, length); - } + if (length > 0) + image_load_increment(State, membuf, length); + + } g_free(membuf); if (State->pixbuf != NULL) gdk_pixbuf_ref(State->pixbuf); @@ -164,16 +218,36 @@ GdkPixbuf *image_load(FILE * f) return State->pixbuf; } -static void DecodeHeader(unsigned char *BFH,unsigned char *BIH, - struct bmp_progressive_state *State) +static void DecodeHeader(unsigned char *BFH, unsigned char *BIH, + struct bmp_progressive_state *State) { - State->Header.width = (BIH[7]<<24)+(BIH[6]<<16)+(BIH[5]<<8)+(BIH[4]); - State->Header.height = (BIH[11]<<24)+(BIH[10]<<16)+(BIH[9]<<8)+(BIH[8]); - State->Header.depth = (BIH[15]<<8)+(BIH[14]);; +/* DumpBIH(BIH);*/ + State->Header.width = + (BIH[7] << 24) + (BIH[6] << 16) + (BIH[5] << 8) + (BIH[4]); + State->Header.height = + (BIH[11] << 24) + (BIH[10] << 16) + (BIH[9] << 8) + (BIH[8]); + State->Header.depth = (BIH[15] << 8) + (BIH[14]);; State->Type = State->Header.depth; /* This may be less trivial someday */ - State->HeaderSize = - ((BFH[13]<<24)+(BFH[12]<<16)+(BFH[11]<<8)+(BFH[10])); + State->HeaderSize = + ((BFH[13] << 24) + (BFH[12] << 16) + (BFH[11] << 8) + + (BFH[10])); + if (State->HeaderSize >= 14 + 40 + 768 + 512) + State->HeaderSize = 14 + 40 + 768 + 512 - 1; + + if ((BIH[16] != 0) || (BIH[17] != 0) || (BIH[18] != 0) + || (BIH[19] != 0)) { + State->Compressed = 1; + } + + if (State->Header.height < 0) { + State->Header.height = -State->Header.height; + State->Header.Negative = 1; + } + if (State->Header.width < 0) { + State->Header.width = -State->Header.width; + State->Header.Negative = 0; + } if (State->Type == 24) State->LineWidth = State->Header.width * 3; @@ -184,11 +258,11 @@ static void DecodeHeader(unsigned char *BFH,unsigned char *BIH, if ((State->Header.width & 7) != 0) State->LineWidth++; } - + /* Pad to a 32 bit boundary */ - if ((State->LineWidth%4)>0) - State->LineWidth=(State->LineWidth%4)*4+4; - + if (((State->LineWidth % 4) > 0) && (State->Compressed == 0)) + State->LineWidth = (State->LineWidth / 4) * 4 + 4; + if (State->LineBuf == NULL) State->LineBuf = g_malloc(State->LineWidth); @@ -198,16 +272,17 @@ static void DecodeHeader(unsigned char *BFH,unsigned char *BIH, if (State->pixbuf == NULL) { State->pixbuf = - gdk_pixbuf_new(ART_PIX_RGB, FALSE, 8, - (gint) State->Header.width, - (gint) State->Header.height); + gdk_pixbuf_new(ART_PIX_RGB, FALSE, 8, + (gint) State->Header.width, + (gint) State->Header.height); + if (State->prepared_func != NULL) /* Notify the client that we are ready to go */ (*State->prepared_func) (State->pixbuf, State->user_data); } - + } /* @@ -228,9 +303,8 @@ image_begin_load(ModulePreparedNotifyFunc prepared_func, context->user_data = user_data; context->HeaderSize = 54; - context->HeaderBuf = g_malloc(14 + - 40 + 768); - /* 768 for the colormap */ + context->HeaderBuf = g_malloc(14 + 40 + 768 + 512); + /* 768 for the colormap */ context->HeaderDone = 0; context->LineWidth = 0; @@ -241,6 +315,7 @@ image_begin_load(ModulePreparedNotifyFunc prepared_func, context->Type = 0; memset(&context->Header, 0, sizeof(struct headerpair)); + memset(&context->compr, 0, sizeof(struct bmp_compression_state)); context->pixbuf = NULL; @@ -264,6 +339,7 @@ void image_stop_load(gpointer data) if (context->LineBuf != NULL) g_free(context->LineBuf); + context->LineBuf = NULL; if (context->HeaderBuf != NULL) g_free(context->HeaderBuf); @@ -280,8 +356,14 @@ static void OneLine24(struct bmp_progressive_state *context) guchar *Pixels; X = 0; - Pixels = context->pixbuf->art_pixbuf->pixels + - context->pixbuf->art_pixbuf->rowstride * (context->Header.height-context->Lines); + if (context->Header.Negative == 0) + Pixels = context->pixbuf->art_pixbuf->pixels + + gdk_pixbuf_get_rowstride(context->pixbuf) * + (context->Header.height - context->Lines - 1); + else + Pixels = context->pixbuf->art_pixbuf->pixels + + gdk_pixbuf_get_rowstride(context->pixbuf) * + context->Lines; while (X < context->Header.width) { Pixels[X * 3 + 0] = context->LineBuf[X * 3 + 2]; Pixels[X * 3 + 1] = context->LineBuf[X * 3 + 1]; @@ -297,16 +379,22 @@ static void OneLine8(struct bmp_progressive_state *context) guchar *Pixels; X = 0; - Pixels = context->pixbuf->art_pixbuf->pixels + - context->pixbuf->art_pixbuf->rowstride * (context->Header.height-context->Lines); + if (context->Header.Negative == 0) + Pixels = context->pixbuf->art_pixbuf->pixels + + gdk_pixbuf_get_rowstride(context->pixbuf) * + (context->Header.height - context->Lines - 1); + else + Pixels = context->pixbuf->art_pixbuf->pixels + + gdk_pixbuf_get_rowstride(context->pixbuf) * + context->Lines; while (X < context->Header.width) { /* The joys of having a BGR byteorder */ Pixels[X * 3 + 0] = - context->HeaderBuf[4*context->LineBuf[X] + 56]; + context->HeaderBuf[4 * context->LineBuf[X] + 56]; Pixels[X * 3 + 1] = - context->HeaderBuf[4*context->LineBuf[X] + 55]; + context->HeaderBuf[4 * context->LineBuf[X] + 55]; Pixels[X * 3 + 2] = - context->HeaderBuf[4*context->LineBuf[X] + 54]; + context->HeaderBuf[4 * context->LineBuf[X] + 54]; X++; } } @@ -317,20 +405,23 @@ static void OneLine1(struct bmp_progressive_state *context) guchar *Pixels; X = 0; - Pixels = context->pixbuf->art_pixbuf->pixels + - context->pixbuf->art_pixbuf->rowstride * (context->Header.height-context->Lines); + if (context->Header.Negative == 0) + Pixels = context->pixbuf->art_pixbuf->pixels + + gdk_pixbuf_get_rowstride(context->pixbuf) * + (context->Header.height - context->Lines - 1); + else + Pixels = context->pixbuf->art_pixbuf->pixels + + gdk_pixbuf_get_rowstride(context->pixbuf) * + context->Lines; while (X < context->Header.width) { int Bit; - - Bit = (context->LineBuf[X/8])>>(7-(X&7)); + + Bit = (context->LineBuf[X / 8]) >> (7 - (X & 7)); Bit = Bit & 1; /* The joys of having a BGR byteorder */ - Pixels[X * 3 + 0] = - context->HeaderBuf[Bit + 32]; - Pixels[X * 3 + 1] = - context->HeaderBuf[Bit + 2 + 32]; - Pixels[X * 3 + 2] = - context->HeaderBuf[Bit + 4 + 32]; + Pixels[X * 3 + 0] = context->HeaderBuf[Bit + 32]; + Pixels[X * 3 + 1] = context->HeaderBuf[Bit + 2 + 32]; + Pixels[X * 3 + 2] = context->HeaderBuf[Bit + 4 + 32]; X++; } } @@ -338,6 +429,10 @@ static void OneLine1(struct bmp_progressive_state *context) static void OneLine(struct bmp_progressive_state *context) { + context->LineDone = 0; + if (context->Lines >= context->Header.height) + return; + if (context->Type == 24) OneLine24(context); if (context->Type == 8) @@ -345,9 +440,6 @@ static void OneLine(struct bmp_progressive_state *context) if (context->Type == 1) OneLine1(context); - context->LineDone = 0; - if (context->Lines > context->Header.height) - return; context->Lines++; if (context->updated_func != NULL) { @@ -376,6 +468,7 @@ gboolean image_load_increment(gpointer data, guchar * buf, guint size) gint BytesToCopy; while (size > 0) { + g_assert(context->LineDone >= 0); if (context->HeaderDone < context->HeaderSize) { /* We still have headerbytes to do */ BytesToCopy = @@ -390,6 +483,124 @@ gboolean image_load_increment(gpointer data, guchar * buf, guint size) buf += BytesToCopy; context->HeaderDone += BytesToCopy; + } else if (context->Compressed) { + /* Compression is done 1 at a time for now */ + switch (context->compr.phase) { + case 0: + if (buf[0] != 0) { /* run count */ + context->compr.phase = 1; + context->compr.RunCount = buf[0]; + } else { /* Escape */ + context->compr.phase = 2; + } + buf++; + size--; + break; + case 1: /* Run count received.... */ + while (context->compr.RunCount > 0) { + BytesToCopy = + context->LineWidth - + context->LineDone; + if (BytesToCopy > + context->compr. + RunCount) BytesToCopy = + context->compr. + RunCount; + if (BytesToCopy > 0) { + memset(context->LineBuf + + context->LineDone, + buf[0], + BytesToCopy); + + context->compr.RunCount -= + BytesToCopy; + context->LineDone += + BytesToCopy; + } + if ( + (context->LineDone >= + context->LineWidth) + && (context->LineWidth > 0)) { + OneLine(context); + } + } + context->compr.phase = 0; + buf++; + size--; + break; + case 2: /* Escape received */ + if (buf[0] == 0) { /* End of line */ + context->compr.phase = 0; + if (context->LineDone > 0) + OneLine(context); + } else if (buf[0] == 1) { /* End of image */ + OneLine(context); + context->compr.phase = 6; + size = 0; + break; + } else if (buf[0] == 2) { /* Cursor displacement */ + context->compr.phase = 4; + } else { + context->compr.phase = 3; + context->compr.RunCount = buf[0]; + } + buf++; + size--; + + break; + case 3: + while ((context->compr.RunCount > 0) + && (size > 0)) { + BytesToCopy = + context->LineWidth - + context->LineDone; + if (BytesToCopy > + context->compr. + RunCount) BytesToCopy = + context->compr. + RunCount; + if (BytesToCopy > size) + BytesToCopy = size; + + if (BytesToCopy > 0) { + memcpy(context->LineBuf + + context->LineDone, + buf, BytesToCopy); + + context->compr.RunCount -= + BytesToCopy; + buf += BytesToCopy; + size -= BytesToCopy; + context->LineDone += + BytesToCopy; + } + if ( + (context->LineDone >= + context->LineWidth) + && (context->LineWidth > 0)) + OneLine(context); + } + if (context->compr.RunCount <= 0) + context->compr.phase = 0; + + break; + case 4: + context->compr.phase = 5; + context->compr.XDelta = buf[0]; + buf++; + size--; + break; + case 5: + context->compr.phase = 0; + context->compr.YDelta = buf[0]; + g_assert(0); /* No implementatio of this yet */ + buf++; + size--; + break; + case 6: + size = 0; + } + } else { /* Pixeldata only */ BytesToCopy = @@ -413,9 +624,9 @@ gboolean image_load_increment(gpointer data, guchar * buf, guint size) } - if (context->HeaderDone >= 14+40) - DecodeHeader(context->HeaderBuf,context->HeaderBuf+14, - context); + if (context->HeaderDone >= 14 + 40) + DecodeHeader(context->HeaderBuf, + context->HeaderBuf + 14, context); } |