<p>Hi Steve,</p>
<p>On 2016-09-26 14:40, Steve Lhomme wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> Similar philosophy as 45b0e1644d8f1812d4b6391a7cefedc9f4d44c6d
 ---</code></pre>
</blockquote>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> -              MessageBox( NULL, L"There was an error while connecting to the Internet.\n"\
 -                                "Thanks a lot for the help anyway.",
 -                                L"Report sending failed", MB_OK);
 +              fprintf(stderr,"There was an error while connecting to the Internet.\n",
 +                      (unsigned long)GetLastError());</code></pre>
</blockquote>
<p>I am personally not a big fan of diagnostics that mention <em>“connecting to the Internet”</em>, mostly because there might be an <em>“internet connection”</em> (whatever that is) but other reasons for not being able to establish a connection to the relevant party.</p>
<p>Examples:</p>
<ul>
<li><p>If there is a firewall blocking connections to the relevant destination, how is this “not being able to connect to the Internet”?</p></li>
<li><p>If the FTP-server is down, it is certainly not a “connecting to the Internet” issue.</p></li>
</ul>
<hr style="height:1px;margin-bottom:20px;background-color:#ddd;color:#ddd" />
<h3 id="unused-argument">Unused argument</h3>
<p>Given that the format-string does not include any place where <code>(unsigned long)GetLastError())</code> is used, either the relevant argument, or include it.</p>
<hr style="height:1px;margin-bottom:20px;background-color:#ddd;color:#ddd" />
<h3 id="further-thoughts">Further thoughts</h3>
<p>I think we are going about this the wrong way, and that it certainly make sense to invoke <code>MessageBox</code> with relevant contents if a bug-report fails to upload (maybe even ask if the user would like to save it somewhere for later access in that case).</p>
<p>In case the upload is successful (which the user certainly expects it to be), I think we can ignore creating a <code>MessageBox</code> since a missing one certainly implies that things went smoothly.</p>