Skip to content

change foo-to-string formatting paths to use Fmt module #64

New issue

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

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

Already on GitHub? # to your account

Closed
graydon opened this issue Jun 24, 2010 · 1 comment
Closed

change foo-to-string formatting paths to use Fmt module #64

graydon opened this issue Jun 24, 2010 · 1 comment
Labels
C-cleanup Category: PRs that clean code up or issues documenting cleanup. E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.

Comments

@graydon
Copy link
Contributor

graydon commented Jun 24, 2010

There are a few too many places where data structures are string-ified (Asm.expr64 or Common.size, for example) that should really be Fmt'ed (streaming pretty-printed). This is both more efficient and makes them more convenient to compose via the "%a" format-string type.

@graydon
Copy link
Contributor Author

graydon commented Jan 27, 2011

WONTFIX (not required for bootstrapping)

oli-obk pushed a commit to oli-obk/rust that referenced this issue Jul 19, 2017
keeperofdakeys pushed a commit to keeperofdakeys/rust that referenced this issue Dec 12, 2017
kazcw pushed a commit to kazcw/rust that referenced this issue Oct 23, 2018
* [ci] check formatting

* [rustfmt] reformat the whole library
eddyb pushed a commit to eddyb/rust that referenced this issue Jun 30, 2020
jaisnan pushed a commit to jaisnan/rust-dev that referenced this issue Aug 22, 2024
This issue was closed.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
C-cleanup Category: PRs that clean code up or issues documenting cleanup. E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.
Projects
None yet
Development

No branches or pull requests

1 participant