author | Mark Lodato <lodatom@gmail.com> | 2010-02-09 04:04:41 (UTC) |
---|---|---|
committer | Mark Lodato <lodatom@gmail.com> | 2010-02-09 04:04:41 (UTC) |
commit | 8aab27f24de70acfbdcee31c634a4b1facf23b92 (patch) (unidiff) | |
tree | 10e5f83a5f0c2a286ec2d1feecf6732f863c9a47 | |
parent | 547a64fbd65de293c290f4e18bbeae958d54aaa7 (diff) | |
download | cgit-8aab27f24de70acfbdcee31c634a4b1facf23b92.zip cgit-8aab27f24de70acfbdcee31c634a4b1facf23b92.tar.gz cgit-8aab27f24de70acfbdcee31c634a4b1facf23b92.tar.bz2 |
html: make all strings 'const char *'
None of the html_* functions modify their argument, so they can all be
'const char *' instead of a simple 'char *'. This removes the need to
cast (or copy) when trying to print a const string.
Signed-off-by: Mark Lodato <lodatom@gmail.com>
-rw-r--r-- | html.c | 34 | ||||
-rw-r--r-- | html.h | 18 |
2 files changed, 26 insertions, 26 deletions
@@ -65,5 +65,5 @@ void html_status(int code, const char *msg, int more_headers) | |||
65 | 65 | ||
66 | void html_txt(char *txt) | 66 | void html_txt(const char *txt) |
67 | { | 67 | { |
68 | char *t = txt; | 68 | const char *t = txt; |
69 | while(t && *t){ | 69 | while(t && *t){ |
@@ -86,5 +86,5 @@ void html_txt(char *txt) | |||
86 | 86 | ||
87 | void html_ntxt(int len, char *txt) | 87 | void html_ntxt(int len, const char *txt) |
88 | { | 88 | { |
89 | char *t = txt; | 89 | const char *t = txt; |
90 | while(t && *t && len--){ | 90 | while(t && *t && len--){ |
@@ -109,5 +109,5 @@ void html_ntxt(int len, char *txt) | |||
109 | 109 | ||
110 | void html_attr(char *txt) | 110 | void html_attr(const char *txt) |
111 | { | 111 | { |
112 | char *t = txt; | 112 | const char *t = txt; |
113 | while(t && *t){ | 113 | while(t && *t){ |
@@ -132,5 +132,5 @@ void html_attr(char *txt) | |||
132 | 132 | ||
133 | void html_url_path(char *txt) | 133 | void html_url_path(const char *txt) |
134 | { | 134 | { |
135 | char *t = txt; | 135 | const char *t = txt; |
136 | while(t && *t){ | 136 | while(t && *t){ |
@@ -148,5 +148,5 @@ void html_url_path(char *txt) | |||
148 | 148 | ||
149 | void html_url_arg(char *txt) | 149 | void html_url_arg(const char *txt) |
150 | { | 150 | { |
151 | char *t = txt; | 151 | const char *t = txt; |
152 | while(t && *t){ | 152 | while(t && *t){ |
@@ -164,3 +164,3 @@ void html_url_arg(char *txt) | |||
164 | 164 | ||
165 | void html_hidden(char *name, char *value) | 165 | void html_hidden(const char *name, const char *value) |
166 | { | 166 | { |
@@ -173,3 +173,3 @@ void html_hidden(char *name, char *value) | |||
173 | 173 | ||
174 | void html_option(char *value, char *text, char *selected_value) | 174 | void html_option(const char *value, const char *text, const char *selected_value) |
175 | { | 175 | { |
@@ -185,3 +185,3 @@ void html_option(char *value, char *text, char *selected_value) | |||
185 | 185 | ||
186 | void html_link_open(char *url, char *title, char *class) | 186 | void html_link_open(const char *url, const char *title, const char *class) |
187 | { | 187 | { |
@@ -259,10 +259,10 @@ char *convert_query_hexchar(char *txt) | |||
259 | 259 | ||
260 | int http_parse_querystring(char *txt, void (*fn)(const char *name, const char *value)) | 260 | int http_parse_querystring(const char *txt_, void (*fn)(const char *name, const char *value)) |
261 | { | 261 | { |
262 | char *t, *value = NULL, c; | 262 | char *t, *txt, *value = NULL, c; |
263 | 263 | ||
264 | if (!txt) | 264 | if (!txt_) |
265 | return 0; | 265 | return 0; |
266 | 266 | ||
267 | t = txt = strdup(txt); | 267 | t = txt = strdup(txt_); |
268 | if (t == NULL) { | 268 | if (t == NULL) { |
@@ -9,10 +9,10 @@ extern void htmlf(const char *format,...); | |||
9 | extern void html_status(int code, const char *msg, int more_headers); | 9 | extern void html_status(int code, const char *msg, int more_headers); |
10 | extern void html_txt(char *txt); | 10 | extern void html_txt(const char *txt); |
11 | extern void html_ntxt(int len, char *txt); | 11 | extern void html_ntxt(int len, const char *txt); |
12 | extern void html_attr(char *txt); | 12 | extern void html_attr(const char *txt); |
13 | extern void html_url_path(char *txt); | 13 | extern void html_url_path(const char *txt); |
14 | extern void html_url_arg(char *txt); | 14 | extern void html_url_arg(const char *txt); |
15 | extern void html_hidden(char *name, char *value); | 15 | extern void html_hidden(const char *name, const char *value); |
16 | extern void html_option(char *value, char *text, char *selected_value); | 16 | extern void html_option(const char *value, const char *text, const char *selected_value); |
17 | extern void html_link_open(char *url, char *title, char *class); | 17 | extern void html_link_open(const char *url, const char *title, const char *class); |
18 | extern void html_link_close(void); | 18 | extern void html_link_close(void); |
@@ -21,3 +21,3 @@ extern int html_include(const char *filename); | |||
21 | 21 | ||
22 | extern int http_parse_querystring(char *txt, void (*fn)(const char *name, const char *value)); | 22 | extern int http_parse_querystring(const char *txt, void (*fn)(const char *name, const char *value)); |
23 | 23 | ||