From ba91afc128960fdb4ac549bd7d8762bf628cea71 Mon Sep 17 00:00:00 2001
From: fiatjaf <fiatjaf@gmail.com>
Date: Sun, 25 May 2025 15:58:18 -0300
Subject: [PATCH] Pointer cannot hold a pointer so it can't be a hidden nil.

---
 nip10/nip10.go   | 12 ++++++------
 nip19/pointer.go | 10 ----------
 2 files changed, 6 insertions(+), 16 deletions(-)

diff --git a/nip10/nip10.go b/nip10/nip10.go
index 380b284..e5dbb4e 100644
--- a/nip10/nip10.go
+++ b/nip10/nip10.go
@@ -2,24 +2,24 @@ package nip10
 
 import "fiatjaf.com/nostr"
 
-func GetThreadRoot(tags nostr.Tags) *nostr.EventPointer {
+func GetThreadRoot(tags nostr.Tags) nostr.Pointer {
 	for _, tag := range tags {
 		if len(tag) >= 4 && tag[0] == "e" && tag[3] == "root" {
 			p, _ := nostr.EventPointerFromTag(tag)
-			return &p
+			return p
 		}
 	}
 
 	firstE := tags.Find("e")
 	if firstE != nil {
 		p, _ := nostr.EventPointerFromTag(firstE)
-		return &p
+		return p
 	}
 
 	return nil
 }
 
-func GetImmediateParent(tags nostr.Tags) *nostr.EventPointer {
+func GetImmediateParent(tags nostr.Tags) nostr.Pointer {
 	var parent nostr.Tag
 	var lastE nostr.Tag
 
@@ -56,14 +56,14 @@ func GetImmediateParent(tags nostr.Tags) *nostr.EventPointer {
 	// that means this event is a direct reply to the parent
 	if parent != nil {
 		p, _ := nostr.EventPointerFromTag(parent)
-		return &p
+		return p
 	}
 
 	if lastE != nil {
 		// if we reached this point and we have at least one "e" we'll use that (the last)
 		// (we don't bother looking for relay or author hints because these clients don't add these anyway)
 		p, _ := nostr.EventPointerFromTag(lastE)
-		return &p
+		return p
 	}
 
 	return nil
diff --git a/nip19/pointer.go b/nip19/pointer.go
index f5ca6b9..207ff31 100644
--- a/nip19/pointer.go
+++ b/nip19/pointer.go
@@ -14,20 +14,10 @@ func EncodePointer(pointer nostr.Pointer) string {
 		} else {
 			return EncodeNprofile(v.PublicKey, v.Relays)
 		}
-	case *nostr.ProfilePointer:
-		if v.Relays == nil {
-			return EncodeNpub(v.PublicKey)
-		} else {
-			return EncodeNprofile(v.PublicKey, v.Relays)
-		}
 	case nostr.EventPointer:
 		return EncodeNevent(v.ID, v.Relays, v.Author)
-	case *nostr.EventPointer:
-		return EncodeNevent(v.ID, v.Relays, v.Author)
 	case nostr.EntityPointer:
 		return EncodeNaddr(v.PublicKey, v.Kind, v.Identifier, v.Relays)
-	case *nostr.EntityPointer:
-		return EncodeNaddr(v.PublicKey, v.Kind, v.Identifier, v.Relays)
 	}
 	return ""
 }