Skip to content

minor fix for typo, indent, link, comment #4714

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

Closed
wants to merge 0 commits into from

Conversation

LiYing2010
Copy link
Contributor

No description provided.

@LiYing2010 LiYing2010 requested a review from a team as a code owner February 23, 2025 13:06
@@ -74,7 +74,6 @@ For more information, see the corresponding sections below.
| [`compilerOpts`](#pass-compiler-and-linker-options) | Compiler options that the cinterop tool passes to the C compiler. |
| [`linkerOpts`](#pass-compiler-and-linker-options) | Linker options that the cinterop tool passes to the linker. |
| [`excludedFunctions`](#ignore-specific-functions) | A space-separated list of function names that should be ignored. |
| `excludedMacros` | |
Copy link
Contributor Author

Choose a reason for hiding this comment

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

JFYI
looks like this property is commented out in line 91
so I think it should be deleted here

Comment on lines 219 to 225
`person` or `person.department` is `null`, the function is not called. Here's the equivalent of the same safe call but with the `if` conditional:

```kotlin
if (person != null && person.department != null) {
person.department.head = managersPool.getManager()
}
```
Copy link
Contributor Author

Choose a reason for hiding this comment

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

JFYI
I moved the Here's the equivalent ...... part here
I think it's better

@@ -27,7 +27,7 @@ See [Update to a new release](releases.md#update-to-a-new-kotlin-version) for de
Kotlin 2.0.20 begins to introduce changes to improve consistency in data classes and replace the Experimental context
receivers feature.

### Data class copy function to have the same visibility as constructor
### Data class copy function will have the same visibility as constructor
Copy link
Contributor Author

Choose a reason for hiding this comment

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

JFYI
it is will have ... in line 8, so I changed it here too

Copy link
Collaborator

Choose a reason for hiding this comment

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

For the reviewer: keep the original title.

Comment on lines 80 to 84
// Branch with only the primary condition. Calls `feedDog()` when `animal` is `Dog`
is Animal.Dog -> animal.feedDog()
// Branch with both primary and guard conditions. Returns `feedCat()` when `Animal` is `Cat` and is not `mouseHunter`
// Branch with both primary and guard conditions. Calls `feedCat()` when `animal` is `Cat` and is not `mouseHunter`
is Animal.Cat if !animal.mouseHunter -> animal.feedCat()
// Returns "Unknown animal" if none of the above conditions match
// Prints "Unknown animal" if none of the above conditions match
Copy link
Contributor Author

Choose a reason for hiding this comment

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

JFYI
I think it should be Calls and Prints instead of Returns here
just like in control-flow.md

@sarahhaggarty sarahhaggarty self-assigned this Feb 26, 2025
@sarahhaggarty
Copy link
Collaborator

Hi @LiYing2010
Thanks for your suggestions! I'm just cross-checking some of your suggestions with various developers. I'll come back to you in a couple of weeks with the final review. Thank you for your patience!

@sarahhaggarty
Copy link
Collaborator

Hi @LiYing2010 !
I'm sorry, but there was an issue with GitHub when I tried to update your commit with a few small modifications. Instead, I added your commit with my minor modifications as part of #4815

Based on feedback from developers, I've accepted all changes except for those in these files:

  • docs/topics/data-analysis/data-analysis-work-with-api.md
  • docs/topics/whatsnew/whatsnew2020.md
  • docs/topics/null-safety.md

And I added some minor modifications to these files:

  • docs/topics/control-flow.md
  • docs/topics/compiler-reference.md

Thank you so much for your contribution! Let me know if you have any questions. Apologies again for the confusion with PRs 🙏

@LiYing2010
Copy link
Contributor Author

@sarahhaggarty
thanks very much for accepting my commit
and I will compare my original commit and the new PR you made, to see what was changed.
thanks again ~~~

@LiYing2010 LiYing2010 deleted the fix_for_typo branch May 2, 2025 08:27
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.

3 participants