Skip to content
  • Nathan Harris's avatar
    Rename ELF extension `map` to `tryConverting` and make internal · c4ff3fc3
    Nathan Harris authored
    Motivation:
    
    As originally pointed out in #48, the `map` prefix alone is not enough context into what the method actually does.
    
    Since it fails the future, and isn't a mapping function, the name should reflect this.
    
    But, this method as-is currently provides little value outside of the client context, so it should not be `public`.
    
    If users provide an adequate use case for having it outside of the package, then it can be made public again.
    
    Modifications:
    
    - Rename: ELF where Value == RESPValue extension `map` to `tryConverting`
    - Change: `tryConverting` from public to internal
    
    Result:
    
    A "problematic" method should no longer be available for API users to hurt themselves with.
    c4ff3fc3