medooze / media-server

WebRTC Media Server
GNU General Public License v2.0
1.36k stars 295 forks source link

feat: Import ArrayView from libwebrtc code #383

Open murillo128 opened 2 weeks ago

mildsunrise commented 2 weeks ago

this is basically std::span, right?

murillo128 commented 2 weeks ago

yes, not sure if it has more goodies

El jue, 7 nov 2024, 16:13, Alba Mendez @.***> escribió:

this is basically std::span, right?

— Reply to this email directly, view it on GitHub https://github.com/medooze/media-server/pull/383#issuecomment-2462489690, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIFN43INBGDBP7SN6QUWELZ7N7TFAVCNFSM6AAAAABRLIU4CCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRSGQ4DSNRZGA . You are receiving this because you authored the thread.Message ID: @.***>

mildsunrise commented 1 week ago

I don't see any extra goodies... if we could instead upgrade to c++20 that would be nice :point_right: :point_left: otherwise if it's too much work then we can merge this

murillo128 commented 1 week ago

fine to fastforward to c++20 if it is supported in all the pipelines