Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Removed dead structures after changes to PartialOrd/Ord derivings.
Remove the `NonMatchesExplode` variant now that no deriving impl uses it. Removed `EnumNonMatching` entirely. Remove now irrelevant `on_matching` field and `HandleNonMatchingEnums` type. Removed unused `EnumNonMatchFunc` type def. Drive-by: revise `EnumNonMatchCollapsedFunc` doc. Made all calls to `expand_enum_method_body` go directly to `build_enum_match_tuple`. Alpha-rename `enum_nonmatch_g` back to `enum_nonmatch_f` to reduce overall diff noise. Inline sole call of `some_ordering_const`. Inline sole call of `ordering_const`. Removed a bunch of code that became dead after the above changes.
- Loading branch information
Showing
14 changed files
with
29 additions
and
352 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
5cee578
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
saw approval from huonw
at pnkfelix@5cee578
5cee578
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
merging pnkfelix/rust/fsk-linear-deriving-partialord = 5cee578 into auto
5cee578
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pnkfelix/rust/fsk-linear-deriving-partialord = 5cee578 merged ok, testing candidate = aeab250
5cee578
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/285
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/286
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/285
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/286
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/288
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/289
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/288
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/294
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/289
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/288
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/291
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/289
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/289
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/290
5cee578
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fast-forwarding master to auto = aeab250