cpp-mp4 icon indicating copy to clipboard operation
cpp-mp4 copied to clipboard

libwebrtc と合わせて使おうとした際にビルドエラーとなる箇所の修正

Open tnoho opened this issue 4 years ago • 3 comments

libwebrtc と合わせて使おうとした際に、 chromium 向けのカスタム libc++ を利用するのですが、その場合においてビルドエラーとなってしまう箇所を修正しました。

変更は以下の二点となります。

  • 使われていない include の削除
  • 宇宙船演算子が暗黙的に削除された際に std::map で必要とされる演算子が不足するため追記

挙動に変化がないように修正しましたので、受け入れていただけますと幸いです。

tnoho avatar May 13 '21 16:05 tnoho

bool BoxType::operator<(const BoxType& r) はおかしいです

m_data[i] > r_data[i] だったら すぐに return false すべきではないでしょうか.

また for loop をまわさずに std::array の < や == を利用すればよいと思います

haruyama avatar May 13 '21 20:05 haruyama

すみません。すごい適当な実装になってました。これで修正できたと思います。

tnoho avatar May 18 '21 15:05 tnoho

@tnoho test と lint がこけてるので確認お願いできますか?おそらく lint は外す必要があると思っています。今回の修正に影響する lint を外してもらって問題ないです。

voluntas avatar May 19 '21 01:05 voluntas