Skip to content
This repository has been archived by the owner on May 4, 2024. It is now read-only.

[table] Fix typo #1042

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[table] Fix typo #1042

wants to merge 1 commit into from

Conversation

jolestar
Copy link
Collaborator

Motivation

Fix typo

Have you read the Contributing Guidelines on pull requests?

yes

Test Plan

(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)

No code change

@jolestar jolestar requested a review from wrwg April 24, 2023 23:31
@@ -77,7 +77,7 @@ module extensions::table {
val
}

/// Returns true iff `table` contains an entry for `key`.
/// Returns true if `table` contains an entry for `key`.
Copy link
Member

Choose a reason for hiding this comment

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

Actually iff is a shortcut for if and only if, which is commonly used in the formal verification world. But I'm OK with changing this if it is confusing ;-)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Got it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants