Tracking issue for the functions for debug escaping `char_escape_debug`

[tbu-]

std::char::escape_debug, std::char::EscapeDebug.

[nikonthethird]

I hope this is the correct spot to mention that all examples of std::char::escape_debug use std::char::escape_default.

https://doc.rust-lang.org/std/primitive.char.html#method.escape_debug

[Mark-Simulacrum]

The above has been fixed; the function is correctly documented. Nominating for libs team discussion as to stabilization, I don't think we're going to get much information out of usage since this is such a niche API. I'm personally not certain I like the name: escape_debug seems odd to me, it's not really anything related to debugging I think.

cc @rust-lang/libs

[tbu-]

debug refers to the fact that it is used for the Debug implementation of String and &str.

I would prefer to call these functions just escape (best would be escape_default, but that one is taken). Could we maybe permanently deprecate escape_default in favor of something like escape_nonascii to make it clear that escaping all non-ASCII code points has nothing to do with "default"?

(I don't suggest ever removing escape_default, just deprecating it.)

[sfackler]

@rfcbot fcp merge

[rfcbot]

Team member @sfackler has proposed to merge this. The next step is review by the rest of the tagged teams:

• @BurntSushi
• @Kimundi
• @alexcrichton
• @aturon
• @brson
• @dtolnay
• @sfackler

Concerns:

• str resolved by Tracking issue for the functions for debug escaping char_escape_debug #35068 (comment)

Once these reviewers reach consensus, this will enter its final comment period. If you spot a major issue that hasn't been raised at any point in this process, please speak up!

See this document for info about what commands tagged team members can give me.

[BurntSushi]

Sorry, but I've lost context on this change and this issue doesn't appear to link to anything. What is the purpose of this new method? Is it just a rename?

[tbu-]

The function escapes characters the way the Debug implementation for str and char does.

[dtolnay]

If we are doing this, I would also like to have a plan for #27791 which includes <str>::escape_debug. Looks like there were concerns about consistency between char's and str's escape_default in #27791 (comment).

@rfcbot concern str

[brson]

Please change this sentence in the docs: "This will escape the characters similar to the Debug implementations of str or char." I assume this is not just similar, it is the implementation of 'Debug'.

[tbu-]

Currently. What if we wanted to change either of these but not the other? escape_default couldn't be changed because the documentation was too concrete.