wireplumber.rs
wireplumber.rs copied to clipboard
Consider renames
- [ ] Consider renaming
new_x()
towith_x()
Should the crate use the more Rusty convention for constructors, for exampleProperties::with_string("x")
vsProperties::new_string("x")
?- See 'grep -rF 'pub fn new_' src/auto/'` for the relevant instances of this.
- [ ] enum const names like
ProxyFeatures::PROXY_FEATURE_BOUND
are redundant - [ ]
LibraryErrorEnum
should be something likeLibraryError
orLibraryErrorCode
- [ ] check for other instances of verbose or redundant identifiers