Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Translatec-api/refcounting.po#559

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
mattwang44 merged 3 commits intopython:3.12frommattwang44:c-api/refcounting
Aug 19, 2023

Conversation

mattwang44
Copy link
Collaborator

@mattwang44mattwang44 commentedAug 6, 2023
edited
Loading

translate some strings that are newly-added in 3.12 doc

previewimage

@mattwang44
Copy link
CollaboratorAuthor

@rockleona Please help review this PR when you're available. Thanks!

rockleona reacted with heart emoji

Comment on lines 257 to 258
"這會在減少 `dst` 舊值的參照計數\\ _之前_\\ 將 `dst` 設定為 `src`,使得因 "
"`dst` 被拆除而觸發的任何程式碼不會相信 `dst` 有指向一個有效物件。"
Copy link
Collaborator

@rockleonarockleonaAug 7, 2023
edited
Loading

Choose a reason for hiding this comment

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

這句好難喔@@
我修改了一下語句看有沒有比較通順

Suggested change
"這會在減少 `dst` 舊值的參照計數\\ _之前_\\ 將 `dst` 設定為 `src`,使得因"
"`dst`被拆除而觸發的任何程式碼不會相信 `dst` 有指向一個有效物件。"
"這會在減少 `dst` 舊值的參照計數\\ _之前_\\ 將 `dst` 設定為 `src`,使得任何程式碼"
"觸發了`dst`被拆除的副作用 (side-effect) 進而不會相信 `dst` 有指向一個有效物件。"

@mattwang44
Copy link
CollaboratorAuthor

翻開覆蓋的陷阱卡

想麻煩@rockleona 參與#568 的討論,一起幫忙看看這樣變更會不會有什麼問題 🙏🏽

rockleona reacted with laugh emoji

@rockleona
Copy link
Collaborator

輪到我的回合了

好的,我待會來去看看

Comment on lines 257 to 258
"這會在減少 `dst` 舊值的參照計數\\ _之前_\\ 將 `dst` 設定為 `src`,使得因 "
"`dst` 被拆除而觸發的任何程式碼不會相信 `dst` 有指向一個有效物件。"
Copy link
CollaboratorAuthor

Choose a reason for hiding this comment

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

  • code 指的應該是被意外觸發的部分(也就是 side effect)
  • 我漏翻 side effect

如果改成這樣呢?

Suggested change
"這會在減少 `dst` 舊值的參照計數\\ _之前_\\ 將 `dst` 設定為 `src`,使得因"
"`dst` 被拆除而觸發的任何程式碼不會相信 `dst` 有指向一個有效物件。"
"這會在減少 `dst` 舊值的參照計數\\ _之前_\\ 將 `dst` 設定為 `src`,使得因"
"`dst` 被拆除而觸發的任何副作用 (side-effect) 之程式碼不會相信 `dst` 是"
"指向一個有效物件。"

@mattwang44
Copy link
CollaboratorAuthor

c-api/refcounting 原文近期有不少變動,最近會再找時間補上這個 PR

@mattwang44
Copy link
CollaboratorAuthor

c-api/refcounting 原文近期有不少變動,最近會再找時間補上這個 PR

已補

Copy link
Collaborator

@rockleonarockleona left a comment

Choose a reason for hiding this comment

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

看完一輪之後只能說請收下我的膝蓋吧(X)
太精準了!

mattwang44 reacted with laugh emoji
Copy link
Collaborator

@rockleonarockleona left a comment

Choose a reason for hiding this comment

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

LGTM!

@mattwang44mattwang44 merged commiteda10c1 intopython:3.12Aug 19, 2023
@mattwang44mattwang44 deleted the c-api/refcounting branchAugust 19, 2023 15:23
beccalzh pushed a commit to beccalzh/python-docs-zh-tw that referenced this pull requestSep 4, 2024
* translate `c-api/refcounting.po`* fix: updated the source strings and their translations* fix(c-api/refcounting): modify based on review comments
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@rockleonarockleonarockleona approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@mattwang44@rockleona

[8]ページ先頭

©2009-2025 Movatter.jp