'last' is better returning an optional

Author: nektro

src/common.zig

@@ -179,7 +179,7 @@ pub fn get_modpath(cachepath: string, d: zigmod.Dep, options: *CollectOptions) !
             if (try extras.doesFolderExist(null, pv)) {
                 return pv;
             }
-            const file_name = try u.last(try u.split(options.alloc, d.path, "/"));
+            const file_name = u.last(try u.split(options.alloc, d.path, "/")).?;
             if (d.version.len > 0) {
                 if (try extras.doesFolderExist(null, pv)) {
                     return pv;

src/util/funcs.zig

@@ -79,10 +79,8 @@ pub fn list_remove(alloc: std.mem.Allocator, input: []string, search: string) ![
     return list.toOwnedSlice();
 }
 
-pub fn last(in: []string) !string {
-    if (in.len == 0) {
-        return error.EmptyArray;
-    }
+pub fn last(in: []string) ?string {
+    if (in.len == 0) return null;
     return in[in.len - 1];
 }