Skip to content

Conversation

@faithandbrave
Copy link
Member

表記揺れによって書き方に迷うところがあったので、一旦書き方をスクリプトで統一しました。

@github-actions
Copy link
Contributor

github-actions bot commented May 22, 2025

プレビュー (HTML) (更新時刻: 2025-05-23 10:38:19 JST)

  • Commit: 16a3e80
  • プレビューの生成には時間がかかります (3~5分)。進捗状況はこちらをご確認ください。

変更記事一覧

※ソース (.md) に直接変更のあった記事を列挙しています。グローバル修飾や変換規則の変更による変化は考慮していません。

@faithandbrave
Copy link
Member Author

問題ありそうなら指摘してください。
数日待って、とくに意見なさそうであればマージします。

@Raclamusi
Copy link
Member

(1),(2),(3) のような、カンマの後にスペースなしのパターンも、スペースありに揃えた方が見栄えが良いかと思います。

@faithandbrave
Copy link
Member Author

@Raclamusi
ご指摘ありがとうございます。そのように直しました。

@faithandbrave
Copy link
Member Author

あ、masterブランチに直接コミットしていまいました。。。まちがった…

@Raclamusi
Copy link
Member

半角スペース区切りも1ページだけありました。

  • ## 備考
    - ランダムアクセスイテレータの範囲を使用する場合、状況によっては(1) (2)のバージョンよりも、(3) (4)を使用する方が効率がよくなることが期待できる。ランダムアクセスイテレータはイテレータ同士の差を定数時間で求められるため、イテレーションを行うことなく2つの範囲の要素数が異なることを検出できるためである

揃えるべきか微妙ですが「と」区切りもあります。

また、範囲の表現についても揺れがありますがどうしましょう?

  • ハイフン: (1)-(4)
  • 波ダッシュ: (1)〜(4)
  • 全角チルダ: (1)~(4)

@faithandbrave
Copy link
Member Author

一旦調べます!
以下は見分けるのが大変なので、ハイフンにしてしまったほうがいい気がしますね。

  • 波ダッシュ: (1)〜(4)
  • 全角チルダ: (1)~(4)

@faithandbrave
Copy link
Member Author

これをハイフンに直しますー

% python3 detect_overload_list_pattern.py
reference/ostream/basic_ostream/op_ostream_free.md:77:5: (1)~(5)
reference/ostream/basic_ostream/op_ostream_free.md:83:5: (6)~(12)
reference/ostream/basic_ostream/op_ostream_free.md:87:5: (13)~(17)
reference/ostream/basic_ostream/op_ostream_free.md:99:5: (18)~(24)
reference/ostream/basic_ostream/op_ostream_free.md:151:5: (3)~(5)
reference/ostream/basic_ostream/op_ostream_free.md:151:14: (15)~(17)
reference/ostream/basic_ostream/op_ostream_free.md:230:4: (6)~(12)
reference/ostream/basic_ostream/op_ostream_free.md:230:14: (18)~(24)
reference/memory/uses_allocator_construction_args.md:38:3: (2)~(6)
reference/memory/uses_allocator_construction_args.md:144:6: (2)~(6)
reference/tuple/tuple/get.md:85:20: (1)〜(4)
reference/tuple/tuple/get.md:97:19: (5)〜(7)
reference/syncstream/basic_osyncstream/op_constructor.md:24:3: (1)〜(4)
reference/syncstream/basic_osyncstream/op_constructor.md:36:3: (1)〜(4)
reference/algorithm/search.md:100:3: (1)〜(4)
reference/algorithm/equal.md:115:34: (1) (2)
reference/algorithm/equal.md:115:51: (3) (4)
reference/regex/regex_token_iterator/op_constructor.md:60:3: (2)~(5)
reference/regex/regex_token_iterator/op_constructor.md:60:85: (3)~(5)
reference/regex/regex_token_iterator/op_constructor.md:65:3: (6)~(9)
reference/regex/regex_token_iterator/op_constructor.md:65:50: (2)~(5)
reference/regex/regex_token_iterator/op_constructor.md:73:9: (2)~(5)
reference/regex/regex_token_iterator/op_constructor.md:74:5: (6)~(9)
reference/regex/regex_token_iterator/op_constructor.md:164:36: (6)~(9)
reference/regex/regex_token_iterator/op_constructor.md:165:21: (6)~(9)
reference/string/basic_string/find.md:69:3: (3) (2)
reference/string/basic_string/rfind.md:65:3: (3) (2)
reference/string/basic_string/find_last_not_of.md:68:3: (3) (2)
reference/string/basic_string/compare.md:127:3: (7) (1)
reference/string/basic_string/find_first_not_of.md:67:3: (3) (2)
reference/string/basic_string/find_last_of.md:66:3: (3) (2)
reference/string/basic_string/find_first_of.md:66:3: (3) (2)
reference/mdspan/layout_stride/mapping/required_span_size.md:53:9: (2) (5)
reference/charconv/from_chars.md:65:3: (2)~(4)
reference/charconv/from_chars.md:83:3: (2)~(5)
reference/charconv/to_chars.md:122:3: (6)~(13)
reference/utility/pair/get.md:105:19: (1)〜(4)
reference/utility/pair/get.md:115:18: (5)〜(12)

@faithandbrave
Copy link
Member Author

一通り直しました

@faithandbrave
Copy link
Member Author

「(1) (2)」は「(1), (2)」に修正。
mdspanのところは意味が異なりそうなので無視してあります。

@Raclamusi
Copy link
Member

basic_string::compare にあったような「区切りなし」も結構ありそうですね……

@faithandbrave
Copy link
Member Author

「(1)(2)」も「(1), (2)」に修正しました〜

@faithandbrave faithandbrave merged commit 3acf283 into master May 26, 2025
18 checks passed
@faithandbrave faithandbrave deleted the fix_overload_number_separator branch May 26, 2025 06:26
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