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 /html.c | |
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 |
1 files changed, 17 insertions, 17 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) { |