Hi,<br>I just commited the first patch (the part without explicit constructor).<br><br>It seems I have a quite good feed back, for making the shared pointer constructor explicit.<br>To add a little more to arguments, TT also made the QSharedDataPointer constructor explicit, so there must be good reasons to do so ;)
<br>So if someone as a real stopper reason to don't make the constructor explicit, say it before the next snapshot (should be on monday next week). After the snapshoot I'll commit only the simple pointer convertions (implicit cast from null, and broken constructor) of my patch.  I will let methods changes left to their maintainers (implicit cast of pointer returning).
<br><br>To explicit it, I will left conversion of:<br>
QShared<Foo> bar() {<br>
   Foo *foo = new Foo();<br>
   return foo;<br>
}<br>
<br>
Because it can be converted as:<br>
QShared<Foo> bar() {<br>

   QShared<Foo> foo(new Foo());<br>

   return foo;<br>

}<br>

<br>
or<br>
QShared<Foo> bar() {<br>

   Foo *foo = new Foo();<br>

   return QShared<Foo>(foo);<br>

}<br>

<br>So I let make the final decision to the maintainers.<br>
<br>Cheers,<br>    Michel<br><br>