<html>

    <head>

      <base href="https://bugs.webkit.org/" />

    </head>

    <body>

      <p>

        <div>

            <b><a class="bz_bug_link 

          bz_status_NEW "

   title="NEW - [Curl] WebSocket platform part is not implemented."

   href="https://bugs.webkit.org/show_bug.cgi?id=144628#c17">Comment # 17</a>

              on <a class="bz_bug_link 

          bz_status_NEW "

   title="NEW - [Curl] WebSocket platform part is not implemented."

   href="https://bugs.webkit.org/show_bug.cgi?id=144628">bug 144628</a>

              from <span class="vcard"><a class="email" href="mailto:peavo&#64;outlook.com" title="peavo&#64;outlook.com">peavo&#64;outlook.com</a>

</span></b>

        <pre>(In reply to <a href="show_bug.cgi?id=144628#c13">comment #13</a>)

<span class="quote">&gt; &gt; MSVC didn't want to convert this, I got the error:

&gt; &gt; 

&gt; &gt; error C2664: 'std::unique_ptr&lt;const char

&gt; &gt; [],std::default_delete&lt;_Ty&gt;&gt;::unique_ptr(const

&gt; &gt; std::unique_ptr&lt;_Ty,std::default_delete&lt;_Ty&gt;&gt; &amp;)' : 

&gt; &gt; cannot convert argument 1 from 'std::unique_ptr&lt;char

&gt; &gt; [],std::default_delete&lt;_Ty&gt;&gt;' to 'std::unique_ptr&lt;const char

&gt; &gt; [],std::default_delete&lt;_Ty&gt;&gt; &amp;&amp;'

&gt; If this really is a bug in MSVC, could you use const_cast to add const

&gt; instead of taking it away?  That would be nicer.

&gt; &gt; I had to add a SocketData constructor since m_sendData.append(SocketData {

&gt; &gt; WTF::move(copy), length }) didn't compile with MSVC. The error I got was:

&gt; &gt; 

&gt; &gt; error C2440 : '&lt;function-style-cast&gt;' : cannot convert from

&gt; &gt; 'initializer-list' to 'WebCore::SocketStreamHandle::SocketData'

&gt; &gt; 1&gt;          No constructor could take the source type, or constructor

&gt; &gt; overload resolution was ambiguous

&gt; Does it work to call the constructor without any initializer-list? (no {})</span >

Thanks for looking into this :)

I removed the const modifier from the unique_ptr member in the last patch to avoid all the casting. I hope this is acceptable :)</pre>

        </div>

      </p>

      <hr>

      <span>You are receiving this mail because:</span>

      <ul>

          <li>You are the assignee for the bug.</li>

      </ul>

    </body>

</html>