Skip to content

Fix misleading comment regarding storage struct reference in 06_ArrayAndStruct (#898)#900

Merged
XdpCs merged 1 commit into
AmazingAng:mainfrom
goingforstudying-ctrl:main
May 10, 2026
Merged

Fix misleading comment regarding storage struct reference in 06_ArrayAndStruct (#898)#900
XdpCs merged 1 commit into
AmazingAng:mainfrom
goingforstudying-ctrl:main

Conversation

@goingforstudying-ctrl
Copy link
Copy Markdown
Contributor

Summary

Fixes #898

Changed the misleading comment "assign a copy of student" to "assign a reference to student" in the 06_ArrayAndStruct lesson.

Details

In Solidity, when you write:

becomes a reference to the original storage variable, not a copy. The previous comment incorrectly stated "assign a copy of student", which could confuse learners about how Solidity storage references work.

Files Changed

All language versions (Chinese, English, Japanese) were updated accordingly.

…AndStruct

Changed 'assign a copy of student' to 'assign a reference to student'
in Chinese, English, and Japanese versions of readme.md and .sol files.

In Solidity,  creates a reference
to the original storage variable, not a copy. The previous comment was
misleading and could confuse learners.

Fixes AmazingAng#898
@goingforstudying-ctrl
Copy link
Copy Markdown
Contributor Author

Hi @AmazingAng and maintainers! 👋

Just a friendly follow-up on this small documentation fix. It's been a few days since the PR was opened, and I wanted to check if there's anything else needed from my side to move it forward.

The change is minimal — just correcting a misleading comment about storage struct references in the tutorial. Happy to make any adjustments if required.

Thank you for your time!

@goingforstudying-ctrl
Copy link
Copy Markdown
Contributor Author

Hi @AmazingAng and maintainers! 👋

Just a friendly follow-up on this small documentation fix. It's been a few days since the PR was opened.

The change corrects a misleading comment about storage struct reference behavior in the 06_ArrayAndStruct chapter. This should help learners better understand how Solidity handles struct references in storage.

Please let me know if any changes are needed!

Thanks for maintaining WTF-Solidity! 🙏

@goingforstudying-ctrl
Copy link
Copy Markdown
Contributor Author

Hi @AmazingAng and maintainers! 👋

Just checking in on this PR — it's been about 5 days since opening and there hasn't been any maintainer feedback yet. Please let me know if:

  1. This PR needs any changes
  2. There's a specific process I should follow
  3. The team is just backlogged and I should wait

Happy to make any adjustments needed!

Thank you for your time.

@XdpCs XdpCs merged commit 363d319 into AmazingAng:main May 10, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Doc] Fix: Misleading comment regarding storage struct reference in 06_ArrayAndStruct

2 participants