<div dir="ltr"><div><div><div><div><div><div>Hye Maarten,<br><br></div>during the last days, I have take some time to look at some improvements to bring to LibreOffice code base on CppCheck report that you've setted up.<br><br></div><div>I don't know the configuration you did, and have just questions/remarks.<br></div><div><br></div>Here are some point I've noted :<br><br></div>1. there is a 'missingConfig' error, recommanding us to run the check with --checkconfig<br></div><br>2. we could minimized the results by removing check on qa sub-directories, which are supposed to be the place for unit tests.<br><br></div>3. if you use a suppression-list to silence some reports, may be this one could be in the git repository so we can use it localy.<br><br></div><div><div><div><div><div><div><div><div><div class="gmail_extra">By the way, thank you for bringing this idea to reality.<br></div><div class="gmail_extra">Actually, my point is if we could bring this warning close to 0, the actually crash-test and coverity status !<br><br></div><div class="gmail_extra">Michaël<br></div><div class="gmail_extra"><br><div class="gmail_quote">2015-01-12 7:16 GMT+01:00 Sand <span dir="ltr"><<a href="mailto:sand.fj.wen@qq.com" target="_blank">sand.fj.wen@qq.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Maarten Hoes,<br> thanks for you replying. Unfortunately, I have read the articles of the link. And more, I have followed this link: <a href="https://wiki.openoffice.org/wiki/Writer" target="_blank">https://wiki.openoffice.org/wiki/Writer</a> , these articles have guided me some how. But I still don't understand some designs in the code, could you give me some tips, i.e. why the message loop in module VCL is designed with several level of "while loop" and some invocations are send to the message queue with Link class? <br><br>Any help will be great!<br><br><div style="text-align:right">Sand Wen<br></div><br>_______________________________________________<br>
LibreOffice mailing list<br>
<a href="mailto:LibreOffice@lists.freedesktop.org">LibreOffice@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/libreoffice" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/libreoffice</a><br>
<br></blockquote></div><br></div></div></div></div></div></div></div></div></div></div>