Remove unused rabbit_definitions:apply_defs/5 #6638
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed Changes
Turns out
rabbit_definitions:apply_defs/5
is not being used anywhere - it's a copy ofapply_defs/4
, but with custom error handling. And it contains a bug where it was trying to import bindings too early:rabbitmq-server/deps/rabbit/src/rabbit_definitions.erl
Line 480 in aec3dcb
apply_defs/4
imports exchanges at exactly the same place.This PR removes this unused function, and a bunch of related exports that are not being used outside of their own module.
Types of Changes
Checklist
CONTRIBUTING.md
document