RESOLVED FIXED272374
Adopt std::span in HTMLParserIdioms.cpp 
https://bugs.webkit.org/show_bug.cgi?id=272374
Summary Adopt std::span in HTMLParserIdioms.cpp
Chris Dumez
Reported 2024-04-08 21:53:04 PDT
Adopt std::span in HTTPParserIdioms.cpp.
Attachments
Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2024-04-08 22:01:52 PDT
Pull request: https://github.com/WebKit/WebKit/pull/27013
EWS
Comment 2 2024-04-09 08:04:45 PDT
Committed 277244@main (0f5f1f492230): <https://commits.webkit.org/277244@main> Reviewed commits have been landed. Closing PR #27013 and removing active labels.
Radar WebKit Bug Importer
Comment 3 2024-04-09 08:05:16 PDT
<rdar://problem/126136439>
Diego Pino
Comment 4 2024-04-09 22:09:42 PDT
GTK-Linux-64-bit-Release-Ubuntu-LTS-Build is failing to build after this change https://build.webkit.org/#/builders/68/builds/33181 ``` /home/buildbot/worker/GTK-Linux-64-bit-Release-Ubuntu-LTS-Build/build/Source/WebCore/html/parser/HTMLParserIdioms.cpp:328:27: error: class template placeholder ‘std::span’ not permitted in this context 328 | StringView timeString(std::span { numberStart, data.data() }); | ```
Diego Pino
Comment 5 2024-04-09 22:46:41 PDT
Re-opening for pull request https://github.com/webkit/webkit/pull/27067
EWS
Comment 6 2024-04-09 22:56:57 PDT
Committed 277288@main (d5ab5f9bf70b): <https://commits.webkit.org/277288@main> Reviewed commits have been landed. Closing PR #27067 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.