rustdoc: clean up and fix ord violations in item sorting

[notriddle]

Based on #128139 with a few minor changes:

• The name sorting function is changed to follow the version sort from the style guide
• the cmp function is redesigned to more obviously make a partial order, by always return cmp() of the same variable as the != above

[notriddle]

CC @orlp

[GuillaumeGomez]

Thanks a lot for the improvements! It's indeed much better.

Just one nit in the commit message: it's Co-authored-by (with only one capital letter) based on https://docs.github.com/en/pull-requests/committing-changes-to-your-project/creating-and-editing-commits/creating-a-commit-with-multiple-authors. Appreciate that you put me as co-author. :)

[notriddle]

Good catch. I've fixed it.

[GuillaumeGomez]

r=me once CI is green.

[notriddle]

@bors r=GuillaumeGomez

[bors]

📌 Commit 5384692 has been approved by GuillaumeGomez

It is now in the queue for this repository.

[GuillaumeGomez]

It's blocking a release so let's increase the priority.

@bors p=10

[bors]

⌛ Testing commit 5384692 with merge c1a6199...

[bors]

☀️ Test successful - checks-actions
Approved by: GuillaumeGomez
Pushing c1a6199 to master...

[rust-timer]

Finished benchmarking commit (c1a6199): comparison URL.

Overall result: ✅ improvements - no action needed

@rustbot label: -perf-regression

Instruction count

This is a highly reliable metric that was used to determine the overall result at the top of this comment.

	mean	range	count
Regressions ❌ (primary)	-	-	0
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-0.7%	[-1.6%, -0.2%]	4
Improvements ✅ (secondary)	-	-	0
All ❌✅ (primary)	-0.7%	[-1.6%, -0.2%]	4

Max RSS (memory usage)

Results (primary -0.2%)

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

	mean	range	count
Regressions ❌ (primary)	2.2%	[2.2%, 2.2%]	1
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-2.6%	[-2.6%, -2.6%]	1
Improvements ✅ (secondary)	-	-	0
All ❌✅ (primary)	-0.2%	[-2.6%, 2.2%]	2

Cycles

This benchmark run did not return any relevant results for this metric.

Binary size

This benchmark run did not return any relevant results for this metric.

Bootstrap: 770.858s -> 772.173s (0.17%)
Artifact size: 328.89 MiB -> 328.95 MiB (0.02%)

[tgross35]

@rustbot label +beta-nominated

As I understand it, we need this in the beta compiler to be able to do the stage0 bump at #128083. That PR is failing at an ICE in 1.81.0-beta.1, which should be fixed by this.

Since it is blocking a step in the release cycle, hopefully we can get this merged before the next dist run.

@GuillaumeGomez please correct me if this isn't accurate.

[GuillaumeGomez]

No this is exactly as stated.