Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ets: Rename and export some types #5649

Merged
merged 2 commits into from
Jan 31, 2022

Conversation

garazdawi
Copy link
Contributor

Rename some ets types to clearer name:

  • access() -> table_access()
  • tab() -> table()
  • type() -> table_type()
  • comp_match_spec() -> compiled_match_spec()

The exported types have their old variants kept for backward compatibility.

The types table_access and table_type have been exported.

For consistency, I also renamed all usage of Tab to Table within the ets module itself.

* access() -> table_access()
* tab() -> table()
* type() -> table_type()
* comp_match_spec() -> compiled_match_spec()

The exported types have their old variants kept for
backward compatibility.
@garazdawi garazdawi added team:VM Assigned to OTP team VM enhancement testing currently being tested, tag is used by OTP internal CI labels Jan 26, 2022
@garazdawi garazdawi self-assigned this Jan 26, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Jan 26, 2022

CT Test Results

       5 files     251 suites   2h 1m 31s ⏱️
4 039 tests 3 816 ✔️ 222 💤 1
4 809 runs  4 532 ✔️ 276 💤 1

For more details on these failures, see this check.

Results for commit 80d7c33.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@garazdawi garazdawi force-pushed the lukas/stdlib/fix-ets-types/GH-4915 branch from 509e353 to 5f6fad9 Compare January 26, 2022 09:10
@garazdawi garazdawi linked an issue Jan 26, 2022 that may be closed by this pull request
@sverker
Copy link
Contributor

sverker commented Jan 26, 2022

git grep "ets:tab(" gave some more hits apart from dets and comp_match_spec is used in dets.

@KennethL
Copy link
Contributor

I think this is good

Copy link
Contributor

@KennethL KennethL left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is good, and as far as I can see there is no incompatibility introduced (old code using the exported types before this change should not get new Dialyzer warnings)

@garazdawi garazdawi force-pushed the lukas/stdlib/fix-ets-types/GH-4915 branch from 5f6fad9 to 80d7c33 Compare January 31, 2022 09:19
@garazdawi garazdawi merged commit 360beed into erlang:master Jan 31, 2022
@garazdawi garazdawi deleted the lukas/stdlib/fix-ets-types/GH-4915 branch January 31, 2022 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement team:VM Assigned to OTP team VM testing currently being tested, tag is used by OTP internal CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

export type ets:type/1
3 participants