summaryrefslogtreecommitdiffabout
authorLars Hjemli <hjemli@gmail.com>2007-06-17 10:58:59 (UTC)
committer Lars Hjemli <hjemli@gmail.com>2007-06-17 10:58:59 (UTC)
commit486c87ec2906c445ca505c877720d56e02aed784 (patch) (unidiff)
tree42d66dc8e9dfee4812f3f37b7bc8deef04d4e78d
parent44947bfcdc0d6e8c7d673bea0538cbf2a182f289 (diff)
downloadcgit-486c87ec2906c445ca505c877720d56e02aed784.zip
cgit-486c87ec2906c445ca505c877720d56e02aed784.tar.gz
cgit-486c87ec2906c445ca505c877720d56e02aed784.tar.bz2
ui-shared: use strcmp() to compare strings
Somehow a couple of pointer comparisions crept into the new repolink() and cgit_tree_link() functions. Fix it. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Diffstat (more/less context) (show whitespace changes)
-rw-r--r--ui-shared.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/ui-shared.c b/ui-shared.c
index 9ab6409..657e8af 100644
--- a/ui-shared.c
+++ b/ui-shared.c
@@ -106,64 +106,64 @@ static char *repolink(char *title, char *class, char *page, char *head,
106 html(" href='"); 106 html(" href='");
107 if (cgit_virtual_root) { 107 if (cgit_virtual_root) {
108 html_attr(cgit_virtual_root); 108 html_attr(cgit_virtual_root);
109 if (cgit_virtual_root[strlen(cgit_virtual_root) - 1] != '/') 109 if (cgit_virtual_root[strlen(cgit_virtual_root) - 1] != '/')
110 html("/"); 110 html("/");
111 html_attr(cgit_repo->url); 111 html_attr(cgit_repo->url);
112 if (cgit_repo->url[strlen(cgit_repo->url) - 1] != '/') 112 if (cgit_repo->url[strlen(cgit_repo->url) - 1] != '/')
113 html("/"); 113 html("/");
114 html(page); 114 html(page);
115 html("/"); 115 html("/");
116 if (path) 116 if (path)
117 html_attr(path); 117 html_attr(path);
118 } else { 118 } else {
119 html(cgit_script_name); 119 html(cgit_script_name);
120 html("?url="); 120 html("?url=");
121 html_attr(cgit_repo->url); 121 html_attr(cgit_repo->url);
122 if (cgit_repo->url[strlen(cgit_repo->url) - 1] != '/') 122 if (cgit_repo->url[strlen(cgit_repo->url) - 1] != '/')
123 html("/"); 123 html("/");
124 html(page); 124 html(page);
125 html("/"); 125 html("/");
126 if (path) 126 if (path)
127 html_attr(path); 127 html_attr(path);
128 delim = "&amp;"; 128 delim = "&amp;";
129 } 129 }
130 if (head && head != cgit_query_head) { 130 if (head && strcmp(head, cgit_query_head)) {
131 html(delim); 131 html(delim);
132 html("h="); 132 html("h=");
133 html_attr(head); 133 html_attr(head);
134 delim = "&amp;"; 134 delim = "&amp;";
135 } 135 }
136 return fmt("%s", delim); 136 return fmt("%s", delim);
137} 137}
138 138
139void cgit_tree_link(char *name, char *title, char *class, char *head, 139void cgit_tree_link(char *name, char *title, char *class, char *head,
140 char *rev, char *path) 140 char *rev, char *path)
141{ 141{
142 char *delim; 142 char *delim;
143 143
144 delim = repolink(title, class, "tree", head, path); 144 delim = repolink(title, class, "tree", head, path);
145 if (rev && rev != cgit_query_head) { 145 if (rev && strcmp(rev, cgit_query_head)) {
146 html(delim); 146 html(delim);
147 html("id="); 147 html("id=");
148 html_attr(rev); 148 html_attr(rev);
149 } 149 }
150 html("'>"); 150 html("'>");
151 html_txt(name); 151 html_txt(name);
152 html("</a>"); 152 html("</a>");
153} 153}
154 154
155void cgit_print_date(time_t secs, char *format) 155void cgit_print_date(time_t secs, char *format)
156{ 156{
157 char buf[64]; 157 char buf[64];
158 struct tm *time; 158 struct tm *time;
159 159
160 time = gmtime(&secs); 160 time = gmtime(&secs);
161 strftime(buf, sizeof(buf)-1, format, time); 161 strftime(buf, sizeof(buf)-1, format, time);
162 html_txt(buf); 162 html_txt(buf);
163} 163}
164 164
165void cgit_print_age(time_t t, time_t max_relative, char *format) 165void cgit_print_age(time_t t, time_t max_relative, char *format)
166{ 166{
167 time_t now, secs; 167 time_t now, secs;
168 168
169 time(&now); 169 time(&now);