Thanks to everyone. I will attempt to make a few corrections and resend.<br><br>Some follow up questions:<br>1) For KProcess it sounds like there is a more current version of the source where this fix makes sense.  Is there another branch I could work  in for this?<br>
<br>2) Can someone give me an example or two where a warning may be intended as a reminder to clean up.... and then what the clean up should be? <br>    There are some cases where some ignored writes are at shutdown (writing a byte to communicate another process) and I am not sure what other action could be taken.  In other cases, the recovery action is not clear -- i.e. log something (how to determine standard way?) and continue?<br>
<br><br><div class="gmail_quote">On Sat, Jun 5, 2010 at 4:55 AM, Rolf Eike Beer <span dir="ltr"><<a href="mailto:kde@opensource.sf-tec.de">kde@opensource.sf-tec.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div><div></div><div class="h5">Xander Moore wrote:<br>
> These are pretty basic fixes, but should get rid of annoying build warnings<br>
> for kdebase.<br>
> I  opened bug 240173 and attached the patch, but apparently that is not the<br>
> best way to get things reviwed and checked in.<br>
><br>
> This is mostly:<br>
> 1) Initializing variables in all  code paths<br>
> 2) Checking return values of write() read() pipe()<br>
> 3) Adding Q_UNUSED(param) for unused parameters<br>
><br>
> Thanks for reviewing and any tips.<br>
<br>
</div></div>If you want to get rid of the return value of a function (e.g. nice() in the<br>
first hunk) try<br>
<br>
  (void) nice(...)<br>
<br>
Just some general notes about that low level C functions:<br>
<br>
Using int as the return code of strlen() will give compiler warnings on some<br>
platforms, e.g. Windows64. Use size_t there. Ok, that one is build only on<br>
Un*x, but anyway. The return code of write() is ssize_t, not int. The return<br>
value of write may be less than the given length without an error, you just<br>
have to send the rest of the buffer later.<br>
<br>
At the end I don't think that this makes any difference here. But this whole<br>
pipe()/write()/whatever stuff should just be thrown away and a KProcess should<br>
be used here. But for that sort of porting it's a bit late, that is 4.6 stuff<br>
IMHO.<br>
<br>
And the floppy ioslave. Hm. I don't think that makes any sense anymore.<br>
<br>
You are silencing some compiler warnings here that remind everybody that some<br>
of this low level fiddling really needs a cleanup. I fear if you shut them up<br>
nobody will touch these things for the next years again.<br>
<br>
Just my thoughts.<br>
<font color="#888888"><br>
Eike<br>
</font></blockquote></div><br>