Date: prev next · Thread: first prev next last
2011 Archives by date, by thread · List index


On 10/08/11 18:44, Jenei Gábor wrote:
Hello Noel,

Well, I just like more if{ <linefeed><code><linefeed> } form even is code is just one line long, but you are right
there are no rules about this, but it is the convention to at least follow the pattern already used in the source file otherwise things look very ugly. So.. if you were creating a new file or modifying source that already has 'if{ <linefeed><code><linefeed> }' type style then it would be ok.
As for the other comments, probably you're right that it's not the best solution, I just haven't understood fully all the objects, as to be honest because of the lot of classes this part of code needs a quite big effort to be modified, as you must understand all the objects before,which can be timeconfusing.
hehe, welcome to libreoffice !!! but anyway you managed to decide to change something, understand the code enough to create a fix, that in itself is quite an achievement, so.. you did good
But I absoloutly agree, that this is not the nicest solution, that's why I just signed it as purposed and not as final patch. Even because it keeps a quite annoying problem, that the overwrite dialog in saving will pop up wrong. So I don't either think that this patch can be pushed in this form.
and attaching the patch to the bug like you did is a *good* thing, if someone later wants to pick this up then you patch will serve as a starting point. So thanks again for that
I just pushed it if someone wants to mind with it. By the way if you say I should correct it, and send it again, I'll do provided that someone is really going to push it afterwards, I just interrupted this job because of the debates about the bug.
that's unfortunate ( but understandable ) I don't know too much about this issue ( and the thread that preceded it was too much hard work to follow )
I am busy with another bug about Base's querywizard.
I am really glad that this experience hasn't discouraged you, your efforts are valuable and useful, don't forget that,

Noel

Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.