X-Git-Url: https://codewiz.org/gitweb?a=blobdiff_plain;f=mware%2Fsprintf.c;h=dacb06c64336bff5510b01a92c83e6b0c0479fc8;hb=48e1c3f2a59544cc07554f5a06fb4bc6f8769b7e;hp=def6d20f035a207968700ce39653f8786f98ed78;hpb=a5175fd44dd7528960b1216182afb4fe96c4664e;p=bertos.git

diff --git a/mware/sprintf.c b/mware/sprintf.c
index def6d20f..dacb06c6 100755
--- a/mware/sprintf.c
+++ b/mware/sprintf.c
@@ -5,42 +5,50 @@
  * This file is part of DevLib - See devlib/README for information.
  * -->
  *
- * \version $Id$
+ * \brief sprintf() implementation based on _formatted_write()
  *
+ * \version $Id$
  * \author Bernardo Innocenti <bernie@develer.com>
- *
- * \brief Simple sprintf() implementation based on _formatted_write()
  */
 
-/*
- * $Log$
- * Revision 1.3  2004/06/27 15:20:26  aleph
- * Change UNUSED() macro to accept two arguments: type and name;
- * Add macro GNUC_PREREQ to detect GCC version during build;
- * Some spacing cleanups and typo fix
- *
- * Revision 1.2  2004/06/03 11:27:09  bernie
- * Add dual-license information.
- *
- * Revision 1.1  2004/05/23 15:43:16  bernie
- * Import mware modules.
- *
- * Revision 1.4  2004/04/03 20:42:27  aleph
- * Remove duplicated defines
- *
- * Revision 1.3  2004/03/24 15:48:53  bernie
- * Remove Copyright messages from Doxygen output
- *
- * Revision 1.2  2004/03/19 16:51:30  bernie
- * Add PROGMEM kludge.
- *
- * Revision 1.1  2004/02/23 09:45:09  aleph
- * Add missing library functions.
- *
- * Revision 1.1  2003/11/13 16:56:37  aleph
- * Add first implementation of dsp firmware
- *
- */
+/*#*
+ *#* $Log$
+ *#* Revision 1.6  2004/11/16 21:15:19  bernie
+ *#* Fix off-by-one bug in [v]snprintf().
+ *#*
+ *#* Revision 1.5  2004/10/03 18:54:36  bernie
+ *#* sprintf(): Fix a serious bug; snprintf(): New function.
+ *#*
+ *#* Revision 1.4  2004/08/25 14:12:09  rasky
+ *#* Aggiornato il comment block dei log RCS
+ *#*
+ *#* Revision 1.3  2004/06/27 15:20:26  aleph
+ *#* Change UNUSED() macro to accept two arguments: type and name;
+ *#* Add macro GNUC_PREREQ to detect GCC version during build;
+ *#* Some spacing cleanups and typo fix
+ *#*
+ *#* Revision 1.2  2004/06/03 11:27:09  bernie
+ *#* Add dual-license information.
+ *#*
+ *#* Revision 1.1  2004/05/23 15:43:16  bernie
+ *#* Import mware modules.
+ *#*
+ *#* Revision 1.4  2004/04/03 20:42:27  aleph
+ *#* Remove duplicated defines
+ *#*
+ *#* Revision 1.3  2004/03/24 15:48:53  bernie
+ *#* Remove Copyright messages from Doxygen output
+ *#*
+ *#* Revision 1.2  2004/03/19 16:51:30  bernie
+ *#* Add PROGMEM kludge.
+ *#*
+ *#* Revision 1.1  2004/02/23 09:45:09  aleph
+ *#* Add missing library functions.
+ *#*
+ *#* Revision 1.1  2003/11/13 16:56:37  aleph
+ *#* Add first implementation of dsp firmware
+ *#*
+ *#*/
 
 #include "compiler.h"
 #include "formatwr.h"
@@ -71,13 +79,13 @@ int PGM_FUNC(vsprintf)(char * str, const char * PGM_ATTR fmt, va_list ap)
 	result = PGM_FUNC(_formatted_write)(
 			fmt, (str ? __str_put_char : __null_put_char), &str, ap);
 
-	/* terminate string */
+	/* Terminate string */
 	*str = '\0';
 
 	return result;
 }
 
-/* FIXME: sprintf_P is incorrectly declared in <stdio.h> */
+
 int PGM_FUNC(sprintf)(char *str, const char * fmt, ...)
 {
 	int result;
@@ -87,8 +95,63 @@ int PGM_FUNC(sprintf)(char *str, const char * fmt, ...)
 	result = PGM_FUNC(vsprintf)(str, fmt, ap);
 	va_end(ap);
 
-	/* terminate string */
-	*str = '\0';
+	return result;
+}
+
+/*!
+ * State information for __sn_put_char()
+ */
+struct __sn_state
+{
+	char *str;
+	size_t len;
+};
+
+/*!
+ * formatted_write() callback used [v]snprintf().
+ */
+static void __sn_put_char(char c, void *ptr)
+{
+	struct __sn_state *state = (struct __sn_state *)ptr;
+
+	if (state->len)
+	{
+		--state->len;
+		*state->str++ = c;
+	}
+}
+
+
+int PGM_FUNC(vsnprintf)(char *str, size_t size, const char * PGM_ATTR fmt, va_list ap)
+{
+	int result = 0;
+
+	/* Make room for traling '\0'. */
+	if (size--)
+	{
+		struct __sn_state state;
+		state.str = str;
+		state.len = size;
+
+		result = PGM_FUNC(_formatted_write)(
+			fmt, (str ? __sn_put_char : __null_put_char), &state, ap);
+
+		/* Terminate string. */
+		*state.str = '\0';
+	}
+
+	return result;
+}
+
+
+int PGM_FUNC(snprintf)(char *str, size_t size, const char * fmt, ...)
+{
+	int result;
+	va_list ap;
+
+	va_start(ap, fmt);
+	result = PGM_FUNC(vsnprintf)(str, size, fmt, ap);
+	va_end(ap);
 
 	return result;
 }