From 96385fde3c84a5d8fe2882b7eeb7becc7231a82f Mon Sep 17 00:00:00 2001 From: Robert Findley Date: Wed, 26 Nov 2025 10:27:32 -0500 Subject: [PATCH] docs: document non-pointer receiver for AudioContent (#660) AudioContent should have had a non-pointer receiver, to be consistent with other content types. Add this to our log of rough edges to reconsider for v2. Fixes #641 --- docs/rough_edges.md | 3 +++ internal/docs/rough_edges.src.md | 3 +++ 2 files changed, 6 insertions(+) diff --git a/docs/rough_edges.md b/docs/rough_edges.md index e7f0a17..41888f8 100644 --- a/docs/rough_edges.md +++ b/docs/rough_edges.md @@ -23,3 +23,6 @@ v2. other notification param types. - Similarly, `ProgressNotificationParams` should probably have been `ProgressParams`. + +- `AudioContent.MarshalJSON` should have had a pointer receiver, to be + consistent with other content types. diff --git a/internal/docs/rough_edges.src.md b/internal/docs/rough_edges.src.md index b5dbadc..fb52391 100644 --- a/internal/docs/rough_edges.src.md +++ b/internal/docs/rough_edges.src.md @@ -22,3 +22,6 @@ v2. other notification param types. - Similarly, `ProgressNotificationParams` should probably have been `ProgressParams`. + +- `AudioContent.MarshalJSON` should have had a pointer receiver, to be + consistent with other content types.