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

[TEST] Update machi.to issue1241 and issue1242 #84

Closed
wants to merge 3 commits into from
Closed

Conversation

ma8ma
Copy link
Owner

@ma8ma ma8ma commented Sep 3, 2023

このPRはテストのためマージしません。

[IDEA] Implement MISC::ends_with(haystack, needle)

haystackの末尾がneedleと一致するかチェックする文字列の操作を実装します。合わせてテストケースを追加し動作を確認します。

新しい規格のC++20には同等の操作を行うメンバー関数std::string::ends_with()std::string_view::ends_with()がありますがJDimが利用する規格はC++17であるため自前で実装します。

[IDEA] Root: Support machi.to URL without subdomain

まちBBSのURLのうち地方名サブドメインが付いてないURLをまちBBSとして扱うように修正します。まちBBSのURLは地方名サブドメインが付いてるタイプとサブドメインが無いタイプの2種類あります。

  • https://tokyo.machi.to/tokyo/
  • https://machi.to/tokyo/

参考文献

https://www.machi.to/bbs/read.cgi/tawara/1416672649/65

[IDEA] Remove link confirmation page for URLs in machi.to threads

まちBBS(machi.to)のスレッドにあるURLからリンク先の確認ページを表示するURLを取り除くように修正します。
まちBBSでofflaw.cgiを使わないモードでスレッドを読み込むと通常リンク(青)をクリックしたときリンク先の確認ページが開かれるようになっていました。

関連のissue: JDimproved#1241, JDimproved#1242

haystackの末尾がneedleと一致するかチェックする文字列の操作を
実装します。合わせてテストケースを追加し動作を確認します。

新しい規格のC++20には同等の操作を行うメンバー関数
`std::string::ends_with()`や`std::string_view::ends_with()`が
ありますがJDimが利用する規格はC++17であるため自前で実装します。
まちBBSのURLのうち地方名サブドメインが付いてないURLをまちBBSとして
扱うように修正します。まちBBSのURLは地方名サブドメインが付いてる
タイプとサブドメインが無いタイプの2種類あります。

例
- `https://tokyo.machi.to/tokyo/`
- `https://machi.to/tokyo/`

参考文献
https://www.machi.to/bbs/read.cgi/tawara/1416672649/65
まちBBS(machi.to)のスレッドにあるURLからリンク先の確認ページを表示する
URLを取り除くように修正します。
まちBBSでofflaw.cgiを使わないモードでスレッドを読み込むと通常リンク(青)
をクリックしたときリンク先の確認ページが開かれるようになっていました。
@ma8ma ma8ma added the archive label Sep 3, 2023
@ma8ma ma8ma changed the title [TEST] [TEST] Update machi.to issue1241 and issue1242 Sep 3, 2023
@ma8ma
Copy link
Owner Author

ma8ma commented Sep 9, 2023

JDimproved#1244, JDimproved#1245, JDimproved#1246 でmasterブランチにマージしたため閉じます。

@ma8ma ma8ma closed this Sep 9, 2023
@ma8ma ma8ma deleted the idea-machi branch September 9, 2023 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant