- From: henbos via GitHub <sysbot+gh@w3.org>
- Date: Thu, 10 Aug 2023 07:16:46 +0000
- To: public-webrtc@w3.org
henbos has just created a new issue for https://github.com/w3c/webrtc-stats: == Stats example uses confusing variable names == https://w3c.github.io/webrtc-stats/#example-of-a-stats-application uses variable names that refer to both RTCStats (`report`) and RTCStatsReport (`currentReport`) as reports. Clearly one is a report and one is a stats object. The former is clearly a `report` and the latter, well up for debate if this should be called `stat` or `stats` or `statsObject`. I don't really care as long as we don't call it a report. Normally I wouldn't care that this example is using silly variable names, but apparently we have duplicated this confusion to both [samples](https://github.com/webrtc/samples/blob/gh-pages/src/content/peerconnection/constraints/js/main.js#L206) and [CLs](https://chromium-review.googlesource.com/c/chromium/src/+/4765344/3/third_party/blink/web_tests/fast/peerconnection/RTCPeerConnection-h264-fallback.html). Please view or discuss this issue at https://github.com/w3c/webrtc-stats/issues/768 using your GitHub account -- Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config
Received on Thursday, 10 August 2023 07:16:48 UTC