Skip to content
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

Update builtin.{txt,jax} (Partly#3) #1340

Merged
merged 10 commits into from
Nov 11, 2023
Merged

Update builtin.{txt,jax} (Partly#3) #1340

merged 10 commits into from
Nov 11, 2023

Conversation

h-east
Copy link
Member

@h-east h-east commented Oct 24, 2023

a~n で始まる関数の変更を反映しました。

Copy link
Contributor

@tsuyoshicho tsuyoshicho left a comment

Choose a reason for hiding this comment

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

細かい確認は不十分ですが、まず確認した内容から

doc/builtin.jax Outdated Show resolved Hide resolved
doc/builtin.jax Outdated Show resolved Hide resolved
doc/builtin.jax Show resolved Hide resolved
doc/builtin.jax Show resolved Hide resolved
doc/builtin.jax Outdated Show resolved Hide resolved
doc/builtin.jax Show resolved Hide resolved
doc/builtin.jax Outdated
ID を取得する。
は |:match|, |:2match|, |:3match| 用に予約されている。3 は
|matchparen| プラグインが使うために予約されている。

Copy link
Contributor

Choose a reason for hiding this comment

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

この空行はないようなので、一応指摘

doc/builtin.jax Show resolved Hide resolved
doc/builtin.jax Outdated Show resolved Hide resolved
@tsuyoshicho tsuyoshicho self-requested a review November 2, 2023 12:08
Copy link
Contributor

@tsuyoshicho tsuyoshicho left a comment

Choose a reason for hiding this comment

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

全部確認しました。
ここでの指摘点の対応が完了したらapproveします
(change requestな指摘は少ないです)

doc/builtin.jax Outdated
Comment on lines 1670 to 1671
< 先頭の桁は1になる。{expr} が無効な場合、または、{winid} のウィ
ンドウが見つからない場合は 0 を返す。
Copy link
Contributor

Choose a reason for hiding this comment

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

comment
「または、」で句読点(読点)をうつと、ちょっと細切れ間がある気がします(書き方次第かとは思うのでcomment)

doc/builtin.jax Outdated
Comment on lines 1674 to 1675
号は行の長さより 1 大きい値を返す。
また、<Cmd> マッピングを使うとカーソルが移動しないので、挿入
Copy link
Contributor

Choose a reason for hiding this comment

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

comment
原文では改行(次のパラグラフではなく、連続あつかい)してない様子

doc/builtin.jax Outdated
Comment on lines 1962 to 1963
{lnum} は |getline()| と同様に使用されるが、{lnum} が 0 の場
合、カーソルはカレント行に留まる。
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
{lnum} は |getline()| と同様に使用されるが、{lnum} が 0 の場
合、カーソルはカレント行に留まる。
{lnum} は |getline()| と同様に使用されるが、{lnum} がゼロの場
合、カーソルはカレント行に留まる。

前出のルールに沿うなら、こちらはゼロ表記かと

doc/builtin.jax Outdated Show resolved Hide resolved
doc/builtin.jax Outdated
Comment on lines 2927 to 2928
を一緒に使うことができる。これは、バッファ名を指定せずに空の文
字列を返す expand('%:h') とは異なる。
Copy link
Contributor

Choose a reason for hiding this comment

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

suggestion:
まず、specifiyじゃないので、指定ではないかなと(一応ですが)

その上でまとめると
「バッファ名を指定せずに空の文字列を返す expand('%:h') とは異なる。」
よりは
「バッファ名が(引数に)なく空の文字列を返す expand('%:h') とは異なる。」
かなと思いました。

カッコのところはニュアンス的で、このままだと使えませんが...

全部を統合するとこの訳でいったほうがよさそう、というのであればOKです

Suggested change
を一緒に使うことができる。これは、バッファ名を指定せずに空の文
字列を返す expand('%:h') とは異なる。
を一緒に使うことができる。これは、バッファ名がなく空の文字列
を返す expand('%:h') とは異なる。

(すみません、作りましたが、改行の調整は必要です)

doc/builtin.jax Outdated
Comment on lines 4803 to 4805
4.0 とも異なる。'ignorecase' はここでは適用されず、常に大文字・
小文字は区別される。
'ignorecase' の値はここでは使用されない。引数 {ic} で示されて
いるように、大文字と小文字は区別される。
Copy link
Contributor

Choose a reason for hiding this comment

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

既存の訳の残りと混在して、文が変になってます

(「'ignorecase' はここでは適用されず、常に大文字・」 は削除かな)

doc/builtin.jax Outdated
"scriptversion" スクリプトのバージョン番号。999999 は |Vim9|
script を表す。
"lnum" "sid" 内の行番号。不明の場合はゼロ。
"nowait" 他の長いマッピングを待たない。
(|:map-<nowait>|)。
"abbr" これが短縮入力 |abbreviations| の場合は True。
Copy link
Contributor

Choose a reason for hiding this comment

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

comment
この True は真かなと思いましたが、どうだろう...

doc/builtin.jax Outdated Show resolved Hide resolved
doc/builtin.jax Outdated Show resolved Hide resolved
h-east and others added 5 commits November 11, 2023 06:53
Co-authored-by: Tsuyoshi CHO <Tsuyoshi.CHO@Gmail.com>
Co-authored-by: Tsuyoshi CHO <Tsuyoshi.CHO@Gmail.com>
Co-authored-by: Tsuyoshi CHO <Tsuyoshi.CHO@Gmail.com>
Copy link
Contributor

@tsuyoshicho tsuyoshicho left a comment

Choose a reason for hiding this comment

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

LGTM!

@h-east h-east merged commit cf5fa75 into master Nov 11, 2023
3 checks passed
@h-east h-east deleted the hh-update-builtin-3 branch November 11, 2023 20:55
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.

None yet

3 participants