Skip to content

Rename ELF extension `map` to `tryConverting` and make internal

Nathan Harris requested to merge rename-map-extension into master

Motivation:

As originally pointed out in #48 (closed), 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.

Merge request reports