Skip to content

Remove the fmt! syntax extension#9919

Merged
bors merged 1 commit intorust-lang:masterfrom
alexcrichton:fmt-begone
Oct 19, 2013
Merged

Remove the fmt! syntax extension#9919
bors merged 1 commit intorust-lang:masterfrom
alexcrichton:fmt-begone

Conversation

@alexcrichton
Copy link
Member

It lived a good life, but its time has come. The groundwork is set for the
official transition after the next snapshot (removal of XXX2 macros)

Copy link
Contributor

Choose a reason for hiding this comment

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

Is it worth replacing this with a syntax extension that .span_errs with something like fmt! has been replaced by format!``? (i.e. keep the fmt! macro around until at least post-0.9.)

Similar to the obsolete syntax stuff, and the removal of auto_encode.

Copy link
Member Author

Choose a reason for hiding this comment

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

Hm, I actually think so. I also wouldn't be opposed to a NOTE at the end of all the errors pointing at the documentation for format!

Copy link
Contributor

Choose a reason for hiding this comment

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

Neither would I.

It lived a good life, but its time has come. The groundwork is set for the
official transition after the next snapshot (removal of XXX2 macros)
@alexcrichton
Copy link
Member Author

Updated to print out that fmt! is deprecated and format! should be used instead.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think 3 would be better. :P

bors added a commit that referenced this pull request Oct 19, 2013
It lived a good life, but its time has come. The groundwork is set for the
official transition after the next snapshot (removal of XXX2 macros)
@bors bors closed this Oct 19, 2013
@bors bors merged commit 29c58c4 into rust-lang:master Oct 19, 2013
@alexcrichton alexcrichton deleted the fmt-begone branch October 19, 2013 18:11
flip1995 pushed a commit to flip1995/rust that referenced this pull request Dec 1, 2022
Link to a list of configurable lints in documentation

changelog: none
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