<div dir="ltr">Hi Dirk.<br><div><div class="gmail_extra"><br><div class="gmail_quote">2015-04-02 18:18 GMT+02:00 Dirk Hohndel <span dir="ltr"><<a href="mailto:dirk@hohndel.org" target="_blank">dirk@hohndel.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
Here's what I'd like to see. Either don't do the datetime rename or if you<br>
do it, have it in its own patch with nothing else mixed in.<br>
Then, if you want to change all the error returns to show these errors to<br>
the user, do that in one patch and nothing else.<br>
As a third patch you can do the split out into a separate function,<br>
explaining any logic changes that you are doing.<br>
<br></blockquote></div><br></div><div class="gmail_extra">Attached are the reworked patches.  I've get rid of the fprintf messages as they should be shown to the user in mainwindow viia report_error(),   erased renaming of datetime as it was pointless, and reworked all other considerations.<br><br></div><div class="gmail_extra">Around the data thing in parse_gasmixes() with the pointer changed to NULL,  explanation is in 0003 patch.  For this point I've added 0007 patch which is not mandatory (but advisable, I think).<br><br></div><div class="gmail_extra">All should apply on master - acaedee159abf<br></div><div class="gmail_extra"><br></div><div class="gmail_extra">Regards.<br><br></div><div class="gmail_extra">Salva.<br></div><div class="gmail_extra"><br></div></div></div>