wireplumber.rs icon indicating copy to clipboard operation
wireplumber.rs copied to clipboard

Consider renames

Open arcnmx opened this issue 10 months ago • 0 comments

  • [ ] Consider renaming new_x() to with_x() Should the crate use the more Rusty convention for constructors, for example Properties::with_string("x") vs Properties::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 like LibraryError or LibraryErrorCode
  • [ ] check for other instances of verbose or redundant identifiers

arcnmx avatar Aug 23 '23 23:08 arcnmx