Skip to content

Comments

service/api: remove variable formatting helper methods#4259

Merged
derekparker merged 1 commit intogo-delve:masterfrom
aarzilli:varformatflags
Feb 16, 2026
Merged

service/api: remove variable formatting helper methods#4259
derekparker merged 1 commit intogo-delve:masterfrom
aarzilli:varformatflags

Conversation

@aarzilli
Copy link
Member

Replace SinglelineStringFormatted, SinglelineStringWithShortTypes and
MultilineString with a single configurable StringWithOptions.

The Variable type was starting to have too many formatting helper methods.

Keep SinglelineString since it's used everywhere.

Replace SinglelineStringFormatted, SinglelineStringWithShortTypes and
MultilineString with a single configurable StringWithOptions.
The Variable type was starting to have too many formatting helper methods.

Keep SinglelineString since it's used everywhere.
Copy link
Member

@derekparker derekparker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@derekparker derekparker merged commit 60b512f into go-delve:master Feb 16, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants