node: move arg check in API declaration for nodeLookupPrefix method

Signed-off-by: Francis Bouvier <[email protected]>

Author: francisbouvier

src/dom/node.zig

@@ -178,6 +178,12 @@ pub const Node = struct {
 
     pub fn _lookupPrefix(self: *parser.Node, namespace: ?[]const u8) ?[]const u8 {
         // TODO: other is not an optional parameter, but can be null.
+        if (namespace == null) {
+            return null;
+        }
+        if (std.mem.eql(u8, namespace.?, "")) {
+            return null;
+        }
         return parser.nodeLookupPrefix(self, namespace);
     }
 

src/netsurf.zig

@@ -383,15 +383,9 @@ pub fn nodeIsSameNode(node: *Node, other: *Node) bool {
     return res;
 }
 
-pub fn nodeLookupPrefix(node: *Node, namespace: ?[]const u8) ?[]const u8 {
-    if (namespace == null) {
-        return null;
-    }
-    if (std.mem.eql(u8, namespace.?, "")) {
-        return null;
-    }
+pub fn nodeLookupPrefix(node: *Node, namespace: []const u8) ?[]const u8 {
     var s: ?*String = undefined;
-    _ = nodeVtable(node).dom_node_lookup_prefix.?(node, stringFromData(namespace.?), &s);
+    _ = nodeVtable(node).dom_node_lookup_prefix.?(node, stringFromData(namespace), &s);
     if (s == null) {
         return null;
     }