KEMBAR78
Fix compare support for old C++ by wx257osn2 · Pull Request #5 · faithandbrave/life_string_view · GitHub
Skip to content

Conversation

wx257osn2
Copy link
Contributor

@wx257osn2 wx257osn2 commented Jul 16, 2024

  • C++17以前だと異なる型同士の比較演算は左右それぞれ定義が必要なので追加
  • operator< とかで文字列リテラルとそのまま比較できなかったので対応
    • ところで本質的には life_string_viewconst char* から直接(暗黙に)構築できないことに起因する問題だと思っていて,こっち側で解決するべきなのかはうーんという感じ(ところでこれはライブラリ本体のdesign decisionだと思っていて,私の一存で決める話でもないので一旦この対応で…)

@faithandbrave faithandbrave merged commit f6c9fcd into faithandbrave:main Jul 16, 2024
@faithandbrave
Copy link
Owner

助かりますー

@wx257osn2 wx257osn2 deleted the fix-compare-support-for-old-cxx branch July 16, 2024 06:07
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.

2 participants