<div dir="ltr"><div><div><div><div><div><div><div><div><div><div>Hey!<br></div>I am hereby attaching my patch which is a work in progress.<br><br></div>Thorsten : Check out <b>Line number 148</b> in sd/source/core/sdtool.cxx where i have mentioned(i.e. in the form of comment) the query i have in mind ,and the question i had asked you on IRC. I hope the surrounding code would make things understandable.<br>
<br></div>Miklos : I have worked on proper identation as well as licence headers(But still if you think there is a scope of improvement,do let me know , There is always a scope of improvement.).<br><br></div>Its a great time working on such an interesting project,but i have some doubts which i would like to clear :<br>
<br></div>1) I am able to code things regarding the STLPropertySet,SfxItemSet,etc. but i am still not able to feel these concepts and specifically the indepth meaning of the Preset! :o ( and yes that is why i have not been able to explain certain functions in the comments properly, I am sure i will do that once i am clear with it .)<br>
</div>So if someone can help me out understand this i am sure i would be able to contribute in a much better and efficient manner as well.<br></div><div><br>2) I wanted to know the exact workflow of the other things that work from the MasterSlide eg:Themes,Backgrounds,etc.I think if i can get the code pointer for it, I would be easily able to know and figure out the loop-hole and get nearer to solving this problem and hence bridge the Custom Animations from The Master Slides.<br>
</div><div>(i am parallely working on the UI Part as well with the UI Team so it would be great if things fall in place simultaneously :) )<br></div>And yes.. My work does not end here,i am continuing to work on the same and will surely make the changes suggested and add the new files i think i would be building in some time. :)<br>
</div><br>Any inputs on code would surely be appreciated.<br><br></div>Hope this finds in good health.<br><br></div>Regards,<br>Janit<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, May 8, 2013 at 6:55 PM, Janit Anjaria <span dir="ltr"><<a href="mailto:janit92@gmail.com" target="_blank">janit92@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div>Hey Miklos!<br><br></div>I suppose i have used the correct header now ,and regarding the identation i dont think there is any need of it here as the header file just declares the functions used.I have made all the changes to the way the comments have to be used and i am sure this learning process is going to help me on the time to come working on this cool project.<br>
</div><div><br>And i am sure any kind of further suggestion will surely polish my skills even on the formal part of the code.<br></div><div>( and yeah i have also started to work on the .cxx file that will complement this header , and i will email it sometime over this week :D )<br>
<br></div><div><br></div><div>Hope this finds you in good health.<br></div>Regards,<br></div>Janit<br></div><div class="gmail_extra"><br><br><div class="gmail_quote"><div><div class="h5">On Tue, May 7, 2013 at 3:01 PM, Miklos Vajna <span dir="ltr"><<a href="mailto:vmiklos@suse.cz" target="_blank">vmiklos@suse.cz</a>></span> wrote:<br>
</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">Hi Janit,<br>
<br>
On Tue, May 07, 2013 at 10:55:53AM +0530, Janit Anjaria <<a href="mailto:janit92@gmail.com" target="_blank">janit92@gmail.com</a>> wrote:<br>
> +/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */<br>
> +/*<br>
> + * This file is part of the LibreOffice project.<br>
> + *<br>
> + * This Source Code Form is subject to the terms of the Mozilla Public<br>
> + * License, v. 2.0. If a copy of the MPL was not distributed with this<br>
> + * file, You can obtain one at <a href="http://mozilla.org/MPL/2.0/" target="_blank">http://mozilla.org/MPL/2.0/</a>.<br>
> + *<br>
> + * This file incorporates work covered by the following license notice:<br>
> + *<br>
> + * Licensed to the Apache Software Foundation (ASF) under one or more<br>
> + * contributor license agreements. See the NOTICE file distributed<br>
> + * with this work for additional information regarding copyright<br>
> + * ownership. The ASF licenses this file to you under the Apache<br>
> + * License, Version 2.0 (the "License"); you may not use this file<br>
> + * except in compliance with the License. You may obtain a copy of<br>
> + * the License at <a href="http://www.apache.org/licenses/LICENSE-2.0" target="_blank">http://www.apache.org/licenses/LICENSE-2.0</a> .<br>
> + */<br>
<br>
Wrong header, for new code, the MPL2 header should be used, see<br>
TEMPLATE.SOURCECODE.HEADER (the license part of that header is only 4<br>
lines).<br>
<br>
> +//This header basically will be complemented by a .cxx file which will be dealing with usage of the declared functions which can be extended so as to use the defined animation object(which we have declared).<br>
> +namespace sd<br>
> +{<br>
> +class View;<br>
<br>
See <a href="https://wiki.documentfoundation.org/Development/Doxygen" target="_blank">https://wiki.documentfoundation.org/Development/Doxygen</a>, you should<br>
use /// to make doxygen pick up documentation. Also: no indentation?<br>
<br>
> +namespace tools<br>
> +{<br>
> +//adding a value to the STLPropertySet.<br>
> +void addValue(STL Property Ser *pSet,Int32 nHandle,const Any& rValue);<br>
<br>
Did you compile-test this "STL Property Ser"?<br>
<span><font color="#888888"><br>
Miklos<br>
</font></span><br></div></div>_______________________________________________<br>
LibreOffice mailing list<br>
<a href="mailto:LibreOffice@lists.freedesktop.org" target="_blank">LibreOffice@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/libreoffice" target="_blank">http://lists.freedesktop.org/mailman/listinfo/libreoffice</a><br>
<br></blockquote></div><br></div>
</blockquote></div><br></div>