Add full support for APIC frames.

* src/frametab.gperf: Support APIC.
* src/idop.c (field_binary_from_string): Accept lowercase
hex digits.
* scheme/idest/batch/setpic.scm: New batch.
* scheme/idest/format/pic.scm: Use named elements.

1 files changed, 5 insertions, 3 deletions

diff --git a/src/idop.c b/src/idop.c
index e10a295..e549bcc 100644
--- a/src/idop.c
+++ b/src/idop.c
@@ -17,6 +17,7 @@
 #include "idest.h"
 #include <file.h>
 #include <signal.h>
+#include <ctype.h>
 
 gl_list_t input_list, output_list, filter_list;
 
@@ -141,8 +142,9 @@ describe_frame(const struct idest_frametab *ft, void *data)
 		for (i = 0; i < ft->qc; i++)
 			n += printf(":%s", ft->qv[i]);
 	}
-	while (++n < DESCR_COLUMN)
+	do
 		putchar(' ');
+	while (n++ < DESCR_COLUMN);
 	
 	printf("%s\n", frametype->description);
 	return 0;
@@ -251,7 +253,7 @@ field_binary_from_string(union id3_field *field, const char *str)
 		unsigned char b;
 		char *p;
 		
-		p = strchr(xdig, *str++);
+		p = strchr(xdig, toupper(*str++));
 		if (!p) {
 			free(field->binary.data);
 			field->binary.length = 0;
@@ -260,7 +262,7 @@ field_binary_from_string(union id3_field *field, const char *str)
 		}
 		b = (p - xdig) << 4;
 
-		p = strchr(xdig, *str++);
+		p = strchr(xdig, toupper(*str++));
 		if (!p) {
 			free(field->binary.data);
 			field->binary.length = 0;