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


Hey Miklos!

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.

And i am sure any kind of further suggestion will surely polish my skills
even on the formal part of the code.
( 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 )


Hope this finds you in good health.
Regards,
Janit


On Tue, May 7, 2013 at 3:01 PM, Miklos Vajna <vmiklos@suse.cz> wrote:

Hi Janit,

On Tue, May 07, 2013 at 10:55:53AM +0530, Janit Anjaria <janit92@gmail.com>
wrote:
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset:
4 -*- */
+/*
+ * This file is part of the LibreOffice project.
+ *
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/.
+ *
+ * This file incorporates work covered by the following license notice:
+ *
+ *   Licensed to the Apache Software Foundation (ASF) under one or more
+ *   contributor license agreements. See the NOTICE file distributed
+ *   with this work for additional information regarding copyright
+ *   ownership. The ASF licenses this file to you under the Apache
+ *   License, Version 2.0 (the "License"); you may not use this file
+ *   except in compliance with the License. You may obtain a copy of
+ *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
+ */

Wrong header, for new code, the MPL2 header should be used, see
TEMPLATE.SOURCECODE.HEADER (the license part of that header is only 4
lines).

+//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).
+namespace sd
+{
+class View;

See https://wiki.documentfoundation.org/Development/Doxygen, you should
use /// to make doxygen pick up documentation. Also: no indentation?

+namespace tools
+{
+//adding a value to the STLPropertySet.
+void addValue(STL Property Ser *pSet,Int32 nHandle,const Any& rValue);

Did you compile-test this "STL Property Ser"?

Miklos

_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Attachment: 0001-fdo-41572-Making-it-possible-to-add-Custom-Animation.patch
Description: Binary data


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.