changed default behavior for parse repo url

Author: aestriplex

src/repo.c

@@ -149,13 +149,6 @@ repository_t parse_repository(const char *line, ssize_t len, unsigned id)
 			   ? empty_str()
 			   : str_init(bracket_open + 1, url_len);
 
-	if (repo.url.len != 0 && get_char(repo.url, repo.url.len - 1) != '/') {
-		str_t postfix = str_init("/", 1);
-		str_t new_url = str_concat(repo.url, postfix);
-		str_free(repo.url);
-		str_free(postfix);
-		repo.url = new_url;
-	}
 	repo.format = (fmt_t) {
 		.commit_url = select_function(repo.url),
 	};

src/utils.c

@@ -35,10 +35,10 @@
 
 #define DEFAULT_EDITOR "vi"
 
-#define GITHUB_URL      "commit/"
-#define GITHUB_URL_SIZE 7
-#define GITLAB_URL      "-/commit/"
-#define GITLAB_URL_SIZE 9
+#define GITHUB_URL      "/commit/"
+#define GITHUB_URL_SIZE 8
+#define GITLAB_URL      "/-/commit/"
+#define GITLAB_URL_SIZE 10
 
 static str_t time_to_full_string(time_t timestamp)
 {

test/test_parse_repository.c

@@ -57,7 +57,7 @@ void test_parse_repository()
 		repository_t repo = parse_repository(line, strlen(line), 0);
 
 		assert_true(str_arr_equals(repo.path, "repo/path"), "Path should be 'repo/path'");
-		assert_true(str_arr_equals(repo.url, "https://example.com/"), "URL should be 'https://example.com/'");
+		assert_true(str_arr_equals(repo.url, "https://example.com"), "URL should be 'https://example.com'");
 
 		str_free(repo.path);
 		str_free(repo.name);
@@ -69,7 +69,7 @@ void test_parse_repository()
 		repository_t repo = parse_repository(line, strlen(line), 0);
 
 		assert_true(repo.path.len == 0, "Path should be empty");
-		assert_true(str_arr_equals(repo.url, "https://example.com/"), "URL should be 'https://example.com/'");
+		assert_true(str_arr_equals(repo.url, "https://example.com"), "URL should be 'https://example.com'");
 
 		str_free(repo.path);
 		str_free(repo.name);
@@ -93,7 +93,7 @@ void test_parse_repository()
 		repository_t repo = parse_repository(line, strlen(line), 0);
 
 		assert_true(str_arr_equals(repo.path, "repo/path"), "Path should be 'repo/path'");
-		assert_true(str_arr_equals(repo.url, "https://example.com/repo[name]/"), "URL should be 'https://example.com/repo[name]/'");
+		assert_true(str_arr_equals(repo.url, "https://example.com/repo[name]"), "URL should be 'https://example.com/repo[name]'");
 
 		str_free(repo.path);
 		str_free(repo.name);
@@ -129,7 +129,7 @@ void test_parse_repository()
 		repository_t repo = parse_repository(line, strlen(line), 0);
 
 		assert_true(str_arr_equals(repo.path, "repo/path"), "Path should be 'repo/path'");
-		assert_true(str_arr_equals(repo.url, "git@github.com:user/repo.git/"), "URL should be 'git@github.com:user/repo.git/'");
+		assert_true(str_arr_equals(repo.url, "git@github.com:user/repo.git"), "URL should be 'git@github.com:user/repo.git'");
 
 		str_free(repo.path);
 		str_free(repo.name);