Skip to content

x/pkgsite: issue title #70387

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
codeimmortal opened this issue Nov 16, 2024 · 4 comments
Closed

x/pkgsite: issue title #70387

codeimmortal opened this issue Nov 16, 2024 · 4 comments
Labels
Milestone

Comments

@codeimmortal
Copy link

What is the URL of the page with the issue?

https://pkg.go.dev/fmt@go1.23.3

What is your user agent?

Chrome/130.0.0.0

Screenshot

Screenshot 2024-11-16 at 9 32 04 AM

What did you do?

If you see the screenshot, you will find these two line.

%x hexadecimal notation (with decimal power of two exponent), e.g. -0x1.23abcp+20
%X upper-case hexadecimal notation, e.g. -0X1.23ABCP+20

if you see e.g -0x1.23abcp+20 , you will find out this is not a proper hexadecimal notation. it should be, d in place of p same for uppercase

%x hexadecimal notation (with decimal power of two exponent), e.g. -0x1.23abcd+20
%X upper-case hexadecimal notation, e.g. -0X1.23ABCD+20

What did you see happen?

If you see the screenshot, you will find these two line.

%x hexadecimal notation (with decimal power of two exponent), e.g. -0x1.23abcp+20
%X upper-case hexadecimal notation, e.g. -0X1.23ABCP+20

there is p in hexadecimal notation, which is wrong notation.

What did you expect to see?

It should be d insted of p, or any other valid hexa character

%x hexadecimal notation (with decimal power of two exponent), e.g. -0x1.23abcd+20
%X upper-case hexadecimal notation, e.g. -0X1.23ABCD+20

@gopherbot gopherbot added this to the Unreleased milestone Nov 16, 2024
@gabyhelp
Copy link

Related Issues

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

@codeimmortal
Copy link
Author

I can raise small PR to correct that.

@codeimmortal
Copy link
Author

codeimmortal commented Nov 16, 2024

Pull request already made. #70388

@ianlancetaylor
Copy link
Member

The text is correct.

See https://go.dev/play/p/ykxYS9yRyJI

@ianlancetaylor ianlancetaylor closed this as not planned Won't fix, can't repro, duplicate, stale Nov 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants