Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[stdlib] Fix SwissTable typos#6595

Open
piperchester wants to merge 4 commits into
modular:mainfrom
piperchester:fix-swiss-table
Open

[stdlib] Fix SwissTable typos#6595
piperchester wants to merge 4 commits into
modular:mainfrom
piperchester:fix-swiss-table

Conversation

@piperchester

@piperchester piperchester commented May 22, 2026

Copy link
Copy Markdown
Contributor

Linked issue

N/A -- trivial.

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • Performance improvement (includes benchmark results below)
  • Documentation update
  • New feature or public API (requires prior proposal or issue approval)
  • Refactor / internal cleanup (no user-visible change)
  • Build, CI, or tooling change

Motivation

Noticed this typo and wanted to fix as in the original CppCon introduction the implementation is "SwissTable."

I think this was probably carried over from https://abseil.io/about/design/swisstables which also seems to have the typo. I looked around and couldn't a more official resource than the original CppCon talk, so went with that as the source of truth.

Super happy to close this out if an undesired change.

What changed

Replaced "Swiss Table" with "SwissTable."

Testing

Verified changes locally.

Checklist

  • The linked issue above has been reviewed by a maintainer and is
    agreed-upon, or this is a trivial fix that does not need prior
    approval
  • PR is small and focused — I've split larger changes into a sequence of
    smaller PRs where possible (see
    pull request sizes)
  • I ran ./bazelw run format to format my changes
  • I added or updated tests to cover my changes
  • If AI tools assisted with this contribution, I have included an
    Assisted-by: trailer in my commit message or this PR description (see
    AI Tool Use Policy)

@github-actions github-actions Bot added mojo-stdlib Tag for issues related to standard library waiting-on-review mojo-docs labels May 22, 2026

### Collections and iterators {#0-26-2-collections}

- `Dict` internals have been replaced with a Swiss Table implementation using

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I didn't see a policy on backporting fixes to release docs, so went ahead and updated this as well.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@piperchester - could you actually remove this releases fix? We generally don't back-fix release doc and in this case using Swiss Table is okay as it's referring to the concept instead of the actual Mojo struct.

@piperchester piperchester May 23, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Absolutely @NathanSWard; updated in b057e2b.

@piperchester piperchester changed the title Fix SwissTable typo [docs] Fix SwissTable typo May 22, 2026
@piperchester

piperchester commented May 22, 2026

Copy link
Copy Markdown
Contributor Author

It's just comment changes, so I wasn't sure to tag [docs] or [stdlib] in the PR title, but err'd on [stdlib] just in case.

@piperchester piperchester changed the title [docs] Fix SwissTable typo [stdlib] Fix SwissTable typo May 22, 2026
@piperchester piperchester changed the title [stdlib] Fix SwissTable typo [stdlib] Fix SwissTable typos May 22, 2026
@piperchester piperchester marked this pull request as ready for review May 22, 2026 16:11
@piperchester piperchester requested review from a team as code owners May 22, 2026 16:11
@piperchester piperchester requested a review from NathanSWard May 23, 2026 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mojo-docs mojo-stdlib Tag for issues related to standard library waiting-on-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants