<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">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>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Janit,<br>
<br>
On Tue, May 07, 2013 at 10:55:53AM +0530, Janit Anjaria <<a href="mailto:janit92@gmail.com">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 class="HOEnZb"><font color="#888888"><br>
Miklos<br>
</font></span><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" target="_blank">http://lists.freedesktop.org/mailman/listinfo/libreoffice</a><br>
<br></blockquote></div><br></div>