From: Carles Pina i Estany <carles@pina.cat>
To: grub-devel@gnu.org
Subject: patch to compile video/readers/* with test
Date: Sat, 26 Dec 2009 11:53:22 +0100 [thread overview]
Message-ID: <20091226105322.GA23263@pina.cat> (raw)
[-- Attachment #1: Type: text/plain, Size: 173 bytes --]
Hi,
Find an attached patch that fix the compilation of video/readers/* when
DEBUG is enabled.
If ok I would commit.
Thanks,
--
Carles Pina i Estany
http://pinux.info
[-- Attachment #2: readers.patch --]
[-- Type: text/x-diff, Size: 4995 bytes --]
=== modified file 'ChangeLog'
--- ChangeLog 2009-12-26 10:01:33 +0000
+++ ChangeLog 2009-12-26 10:42:57 +0000
@@ -1,5 +1,23 @@
2009-12-27 Carles Pina i Estany <carles@pina.cat>
+ * video/readers/jpeg.c (cmd): Declare.
+ (grub_cmd_jpegtest): Use `grub_command_t' type.
+ (GRUB_MOD_INIT): Fix arguments passed to `grub_register_command'. Assign
+ to `cmd'.
+ (GRUB_MOD_FINI): Use `cmd' to unregister.
+ * video/readers/png.c (cmd): Declare.
+ (grub_cmd_pngtest): Use `grub_command_t' type.
+ (GRUB_MOD_INIT): Fix arguments passed to `grub_register_command'. Assign
+ to `cmd'.
+ (GRUB_MOD_FINI): Use `cmd' to unregister.
+ * video/readers/tga.c (cmd): Declare.
+ (grub_cmd_tgatest): Use `grub_command_t' type.
+ (GRUB_MOD_INIT): Fix arguments passed to `grub_register_command'. Assign
+ to `cmd'.
+ (GRUB_MOD_FINI): Use `cmd' to unregister.
+
+2009-12-27 Carles Pina i Estany <carles@pina.cat>
+
* efiemu/main.c (GRUB_MOD_INIT): Fix capitalizations and/or full
stops.
* kern/corecmd.c (grub_register_core_commands): Likewise.
=== modified file 'video/readers/jpeg.c'
--- video/readers/jpeg.c 2009-12-25 23:50:59 +0000
+++ video/readers/jpeg.c 2009-12-26 10:35:09 +0000
@@ -54,6 +54,10 @@ static const grub_uint8_t jpeg_zigzag_or
53, 60, 61, 54, 47, 55, 62, 63
};
+#ifdef JPEG_DEBUG
+static grub_command_t cmd;
+#endif
+
typedef int jpeg_data_unit_t[64];
struct grub_jpeg_data
@@ -695,8 +699,8 @@ grub_video_reader_jpeg (struct grub_vide
#if defined(JPEG_DEBUG)
static grub_err_t
-grub_cmd_jpegtest (struct grub_arg_list *state __attribute__ ((unused)),
- int argc, char **args)
+grub_cmd_jpegtest (grub_command_t cmd __attribute__ ((unused)),
+ int argc, char **args)
{
struct grub_video_bitmap *bitmap = 0;
@@ -730,16 +734,16 @@ GRUB_MOD_INIT (video_reader_jpeg)
grub_video_bitmap_reader_register (&jpg_reader);
grub_video_bitmap_reader_register (&jpeg_reader);
#if defined(JPEG_DEBUG)
- grub_register_command ("jpegtest", grub_cmd_jpegtest,
- GRUB_COMMAND_FLAG_BOTH, "FILE",
- "Tests loading of JPEG bitmap.", 0);
+ cmd = grub_register_command ("jpegtest", grub_cmd_jpegtest,
+ "FILE",
+ "Tests loading of JPEG bitmap.");
#endif
}
-GRUB_MOD_FINI (video_reader_jpeg)
+GRUB_MOD_FINI (grub_cmd_jpegtest)
{
#if defined(JPEG_DEBUG)
- grub_unregister_command ("jpegtest");
+ grub_unregister_command (cmd);
#endif
grub_video_bitmap_reader_unregister (&jpeg_reader);
grub_video_bitmap_reader_unregister (&jpg_reader);
=== modified file 'video/readers/png.c'
--- video/readers/png.c 2009-12-25 23:50:59 +0000
+++ video/readers/png.c 2009-12-26 10:40:10 +0000
@@ -73,6 +73,10 @@
#define DEFLATE_HUFF_LEN 16
+#ifdef PNG_DEBUG
+static grub_command_t cmd;
+#endif
+
struct huff_table
{
int *values, *maxval, *offset;
@@ -866,7 +870,7 @@ grub_video_reader_png (struct grub_video
#if defined(PNG_DEBUG)
static grub_err_t
-grub_cmd_pngtest (struct grub_arg_list *state __attribute__ ((unused)),
+grub_cmd_pngtest (grub_command_t cmd __attribute__ ((unused)),
int argc, char **args)
{
struct grub_video_bitmap *bitmap = 0;
@@ -894,16 +898,16 @@ GRUB_MOD_INIT (video_reader_png)
{
grub_video_bitmap_reader_register (&png_reader);
#if defined(PNG_DEBUG)
- grub_register_command ("pngtest", grub_cmd_pngtest,
- GRUB_COMMAND_FLAG_BOTH, "FILE",
- "Tests loading of PNG bitmap.", 0);
+ cmd = grub_register_command ("pngtest", grub_cmd_pngtest,
+ "FILE",
+ "Tests loading of PNG bitmap.");
#endif
}
GRUB_MOD_FINI (video_reader_png)
{
#if defined(PNG_DEBUG)
- grub_unregister_command ("pngtest");
+ grub_unregister_command (cmd);
#endif
grub_video_bitmap_reader_unregister (&png_reader);
}
=== modified file 'video/readers/tga.c'
--- video/readers/tga.c 2009-12-25 23:50:59 +0000
+++ video/readers/tga.c 2009-12-26 10:39:37 +0000
@@ -29,6 +29,7 @@
#if defined(TGA_DEBUG)
#define dump_int_field(x) grub_printf( #x " = %d (0x%04x)\n", x, x);
+static grub_command_t cmd;
#endif
enum
@@ -452,7 +453,7 @@ grub_video_reader_tga (struct grub_video
#if defined(TGA_DEBUG)
static grub_err_t
-grub_cmd_tgatest (struct grub_arg_list *state __attribute__ ((unused)),
+grub_cmd_tgatest (grub_command_t cmd __attribute__ ((unused)),
int argc, char **args)
{
struct grub_video_bitmap *bitmap = 0;
@@ -480,15 +481,15 @@ GRUB_MOD_INIT(video_reader_tga)
{
grub_video_bitmap_reader_register (&tga_reader);
#if defined(TGA_DEBUG)
- grub_register_command ("tgatest", grub_cmd_tgatest, GRUB_COMMAND_FLAG_BOTH,
- "FILE", "Tests loading of TGA bitmap.", 0);
+ cmd = grub_register_command ("tgatest", grub_cmd_tgatest,
+ "FILE", "Tests loading of TGA bitmap.");
#endif
}
GRUB_MOD_FINI(video_reader_tga)
{
#if defined(TGA_DEBUG)
- grub_unregister_command ("tgatest");
+ grub_unregister_command (cmd);
#endif
grub_video_bitmap_reader_unregister (&tga_reader);
}
next reply other threads:[~2009-12-26 10:53 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-12-26 10:53 Carles Pina i Estany [this message]
2009-12-26 10:55 ` patch to compile video/readers/* with debug Carles Pina i Estany
2009-12-26 11:07 ` patch to compile video/readers/* with test Carles Pina i Estany
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20091226105322.GA23263@pina.cat \
--to=carles@pina.cat \
--cc=grub-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.