[PATCH 09/11] lib: fwts_log: constify function arguments where appropriate

Colin King colin.king at canonical.com
Tue Oct 16 19:20:46 UTC 2012


From: Colin Ian King <colin.king at canonical.com>

Signed-off-by: Colin Ian King <colin.king at canonical.com>
---
 src/lib/include/fwts_log.h |    4 ++--
 src/lib/src/fwts_log.c     |   12 ++++++------
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/lib/include/fwts_log.h b/src/lib/include/fwts_log.h
index 35f2801..5406897 100644
--- a/src/lib/include/fwts_log.h
+++ b/src/lib/include/fwts_log.h
@@ -121,7 +121,7 @@ extern fwts_log_ops fwts_log_html_ops;
 extern fwts_log_field fwts_log_filter;
 extern const char *fwts_log_format;
 
-fwts_log *fwts_log_open(const char* owner, const char *name, const char *mode, fwts_log_type);
+fwts_log *fwts_log_open(const char* owner, const char *name, const char *mode, const fwts_log_type);
 int       fwts_log_close(fwts_log *log);
 int       fwts_log_printf(fwts_log *log, const fwts_log_field field, const fwts_log_level level, const char *status, const char *label, const char *prefix, const char *fmt, ...)
 	__attribute__((format(printf, 7, 8)));
@@ -143,7 +143,7 @@ char	 *fwts_log_field_to_str_upper(const fwts_log_field field);
 void	  fwts_log_set_line_width(const int width);
 void	  fwts_log_section_begin(fwts_log *log, const char *name);
 void	  fwts_log_section_end(fwts_log *log);
-char     *fwts_log_get_filenames(const char *filename, fwts_log_type type);
+char     *fwts_log_get_filenames(const char *filename, const fwts_log_type type);
 fwts_log_filename_type fwts_log_get_filename_type(const char *name);
 
 static inline int fwts_log_type_count(fwts_log_type type)
diff --git a/src/lib/src/fwts_log.c b/src/lib/src/fwts_log.c
index 6ed32c1..63983d5 100644
--- a/src/lib/src/fwts_log.c
+++ b/src/lib/src/fwts_log.c
@@ -43,7 +43,7 @@ const char *fwts_log_format = "";
  *  fwts_log_set_line_width()
  * 	set width of a log
  */
-void fwts_log_set_line_width(int width)
+void fwts_log_set_line_width(const int width)
 {
 	if ((width >= 80) && (width <= 1024))
 		log_line_width = width;
@@ -310,7 +310,7 @@ void fwts_log_set_format(const char *str)
  *  fwts_log_type_filename_suffix()
  *	return a filename suffix on a given log type
  */
-static char *fwts_log_type_filename_suffix(fwts_log_type type)
+static char *fwts_log_type_filename_suffix(const fwts_log_type type)
 {
 	switch (type) {
 	case LOG_TYPE_JSON:
@@ -330,7 +330,7 @@ static char *fwts_log_type_filename_suffix(fwts_log_type type)
  *  fwts_log_filename_new_suffix()
  *	return the log name with suffix based on log type
  */
-static char *fwts_log_filename(const char *filename, fwts_log_type type)
+static char *fwts_log_filename(const char *filename, const fwts_log_type type)
 {
 	char *ptr;
 	char *new_name;
@@ -534,7 +534,7 @@ void fwts_log_section_end(fwts_log *log)
  *  fwts_log_get_ops()
  *	return log ops basedon log type
  */
-static fwts_log_ops *fwts_log_get_ops(fwts_log_type type)
+static fwts_log_ops *fwts_log_get_ops(const fwts_log_type type)
 {
 	switch (type) {
 	case LOG_TYPE_JSON:
@@ -569,7 +569,7 @@ fwts_log_filename_type fwts_log_get_filename_type(const char *filename)
  *  fwts_log_filenames()
  *	return string of all the log filenames that will be used
  */
-char *fwts_log_get_filenames(const char *filename, fwts_log_type type)
+char *fwts_log_get_filenames(const char *filename, const fwts_log_type type)
 {
 	unsigned int i;
 	char *filenames = NULL;
@@ -614,7 +614,7 @@ fwts_log *fwts_log_open(
 	const char *owner,	/* Creator of the log */
 	const char *filename,	/* Log file name */
 	const char *mode,	/* open mode, see fopen() modes */
-	fwts_log_type type)	/* Log type */
+	const fwts_log_type type) /* Log type */
 {
 	fwts_log *newlog;
 	unsigned int i;
-- 
1.7.10.4




More information about the fwts-devel mailing list