summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRafaël Carré <rafael.carre@gmail.com>2010-09-20 08:55:45 +0000
committerRafaël Carré <rafael.carre@gmail.com>2010-09-20 08:55:45 +0000
commit5a98ad2d7fd9980b6adc0f6383f1dc19560858dc (patch)
tree479160253c65a109f04e15997b35c6320c107b37
parentcc6ef19dd97a5dbbd901f9dbede5f65ef60c4a6f (diff)
downloadrockbox-5a98ad2d7fd9980b6adc0f6383f1dc19560858dc.tar.gz
rockbox-5a98ad2d7fd9980b6adc0f6383f1dc19560858dc.zip
format() (and its alias vuprintf) return values are uncheck -> void
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@28119 a1c6a512-1295-4272-9138-f99709370657
-rw-r--r--firmware/common/format.c10
-rw-r--r--firmware/include/format.h4
-rw-r--r--firmware/libc/sprintf.c6
3 files changed, 8 insertions, 12 deletions
diff --git a/firmware/common/format.c b/firmware/common/format.c
index 240dd1ebd5..afe1010cfb 100644
--- a/firmware/common/format.c
+++ b/firmware/common/format.c
@@ -29,7 +29,7 @@
29 29
30static const char hexdigit[] = "0123456789ABCDEF"; 30static const char hexdigit[] = "0123456789ABCDEF";
31 31
32int format( 32void format(
33 /* call 'push()' for each output letter */ 33 /* call 'push()' for each output letter */
34 int (*push)(void *userp, unsigned char data), 34 int (*push)(void *userp, unsigned char data),
35 void *userp, 35 void *userp,
@@ -220,7 +220,6 @@ int format(
220 else 220 else
221 ok=push(userp, ch); 221 ok=push(userp, ch);
222 } 222 }
223 return ok; /* true means good */
224} 223}
225 224
226struct for_fprintf { 225struct for_fprintf {
@@ -244,7 +243,6 @@ static int fprfunc(void *pr, unsigned char letter)
244 243
245int fdprintf(int fd, const char *fmt, ...) 244int fdprintf(int fd, const char *fmt, ...)
246{ 245{
247 bool ok;
248 va_list ap; 246 va_list ap;
249 struct for_fprintf fpr; 247 struct for_fprintf fpr;
250 248
@@ -252,13 +250,13 @@ int fdprintf(int fd, const char *fmt, ...)
252 fpr.bytes=0; 250 fpr.bytes=0;
253 251
254 va_start(ap, fmt); 252 va_start(ap, fmt);
255 ok = format(fprfunc, &fpr, fmt, ap); 253 format(fprfunc, &fpr, fmt, ap);
256 va_end(ap); 254 va_end(ap);
257 255
258 return fpr.bytes; /* return 0 on error */ 256 return fpr.bytes; /* return 0 on error */
259} 257}
260 258
261int vuprintf(int (*push)(void *userp, unsigned char data), void *userp, const char *fmt, va_list ap) 259void vuprintf(int (*push)(void *userp, unsigned char data), void *userp, const char *fmt, va_list ap)
262{ 260{
263 return format(push, userp, fmt, ap); 261 format(push, userp, fmt, ap);
264} 262}
diff --git a/firmware/include/format.h b/firmware/include/format.h
index 6a00574644..30a072aca8 100644
--- a/firmware/include/format.h
+++ b/firmware/include/format.h
@@ -22,7 +22,7 @@
22#ifndef __FORMAT_H__ 22#ifndef __FORMAT_H__
23#define __FORMAT_H__ 23#define __FORMAT_H__
24 24
25int format( 25void format(
26 /* call 'push()' for each output letter */ 26 /* call 'push()' for each output letter */
27 int (*push)(void *userp, unsigned char data), 27 int (*push)(void *userp, unsigned char data),
28 void *userp, 28 void *userp,
@@ -31,7 +31,7 @@ int format(
31 31
32/* callback function is called for every output character (byte) with userp and 32/* callback function is called for every output character (byte) with userp and
33 * should return 0 when ch is a char other than '\0' that should stop printing */ 33 * should return 0 when ch is a char other than '\0' that should stop printing */
34int vuprintf(int (*push)(void *userp, unsigned char data), 34void vuprintf(int (*push)(void *userp, unsigned char data),
35 void *userp, const char *fmt, va_list ap); 35 void *userp, const char *fmt, va_list ap);
36 36
37#endif /* __FORMAT_H__ */ 37#endif /* __FORMAT_H__ */
diff --git a/firmware/libc/sprintf.c b/firmware/libc/sprintf.c
index b02f5a2fae..18e2ce6fd2 100644
--- a/firmware/libc/sprintf.c
+++ b/firmware/libc/sprintf.c
@@ -57,7 +57,6 @@ static int sprfunc(void *ptr, unsigned char letter)
57 57
58int snprintf(char *buf, size_t size, const char *fmt, ...) 58int snprintf(char *buf, size_t size, const char *fmt, ...)
59{ 59{
60 bool ok;
61 va_list ap; 60 va_list ap;
62 struct for_snprintf pr; 61 struct for_snprintf pr;
63 62
@@ -66,7 +65,7 @@ int snprintf(char *buf, size_t size, const char *fmt, ...)
66 pr.max = size; 65 pr.max = size;
67 66
68 va_start(ap, fmt); 67 va_start(ap, fmt);
69 ok = format(sprfunc, &pr, fmt, ap); 68 format(sprfunc, &pr, fmt, ap);
70 va_end(ap); 69 va_end(ap);
71 70
72 /* make sure it ends with a trailing zero */ 71 /* make sure it ends with a trailing zero */
@@ -77,14 +76,13 @@ int snprintf(char *buf, size_t size, const char *fmt, ...)
77 76
78int vsnprintf(char *buf, size_t size, const char *fmt, va_list ap) 77int vsnprintf(char *buf, size_t size, const char *fmt, va_list ap)
79{ 78{
80 bool ok;
81 struct for_snprintf pr; 79 struct for_snprintf pr;
82 80
83 pr.ptr = (unsigned char *)buf; 81 pr.ptr = (unsigned char *)buf;
84 pr.bytes = 0; 82 pr.bytes = 0;
85 pr.max = size; 83 pr.max = size;
86 84
87 ok = format(sprfunc, &pr, fmt, ap); 85 format(sprfunc, &pr, fmt, ap);
88 86
89 /* make sure it ends with a trailing zero */ 87 /* make sure it ends with a trailing zero */
90 pr.ptr[(pr.bytes < pr.max) ? 0 : -1] = '\0'; 88 pr.ptr[(pr.bytes < pr.max) ? 0 : -1] = '\0';