Good question. Changes I requested in the script today do not affect these generated files so I propose to push them.<br><br>On Friday, 9 May 2014, Wojciech Kosowicz <<a href="mailto:pcellix@gmail.com">pcellix@gmail.com</a>> wrote:<br>
> This is an automatically generated e-mail. To reply, visit: <a href="https://git.reviewboard.kde.org/r/117956/">https://git.reviewboard.kde.org/r/117956/</a><br>><br>> On May 9th, 2014, 2:16 p.m. UTC, Jarosław Staniek wrote:<br>
><br>> Good work! Please push both to calligra/2.8 and master.<br>><br>> On May 9th, 2014, 2:39 p.m. UTC, Wojciech Kosowicz wrote:<br>><br>> Thanks, before pushing I have a question regarding code formatting. I noticed that in the previous edition there were used 4 spaces. Now there's 2 tabs. I have not modified any part of the script regarding coding style. Still I want to ensure that is okay to push the way it is. So, is it okay?<br>
><br>> if any coding style required then can be introduced easily, still I'm not 100% if to introduce and rerun script as this is how the script was originally accepted<br>><br>> - Wojciech<br>><br>> On May 8th, 2014, 1:04 p.m. UTC, Wojciech Kosowicz wrote:<br>
><br>> Review request for Calligra and Jarosław Staniek.<br>> By Wojciech Kosowicz.<br>><br>> Updated May 8, 2014, 1:04 p.m.<br>><br>> Repository: calligra<br>><br>> Description<br>><br>> I downloaded sqlite3080403 sources and using script sql_keywords got a generated list of specific sqlite keywords. There are new words in consideration to the previous version. sql_keywords I had to modify as it does not work in it current form (after finishing all changes for all the three databases I will upload modified version to review).<br>
><br>> Diffs<br>><br>> libs/db/drivers/sqlite/sqlitekeywords.cpp (735f446)<br>><br>> View Diff<br><br>-- <br>regards / pozdrawiam, Jaroslaw Staniek<br> Kexi & Calligra & KDE | <a href="http://calligra.org/kexi" target="_blank">http://calligra.org/kexi</a> | <a href="http://kde.org" target="_blank">http://kde.org</a><br>
 Qt for Tizen | <a href="http://qt-project.org/wiki/Tizen" target="_blank">http://qt-project.org/wiki/Tizen</a><br> Qt Certified Specialist | <a href="http://www.linkedin.com/in/jstaniek" target="_blank">http://www.linkedin.com/in/jstaniek</a><br>
<br>