Review Request 118942: support for gogebra's tools in kig ( GSoC 14 )
Aniket Anvit
seeanvit at gmail.com
Sat Jul 19 16:54:10 UTC 2014
> On July 19, 2014, 1:25 a.m., David Narváez wrote:
> > Can you confirm the imported tools work as expected? I imported two tools and they are loaded OK (good names, good descriptions) but the tools themselves are probably not working (although I can't remember what my tools did :P). I have an argument against dropping the ReadingSection state, but lets not worry about that for now, we can add it later if needed.
> >
> > Re: GCC, don't bother with GCC 3.4, that's very old. Let's keep track of GCC >=4.5. If it helps, I have GCC 4.7 to 4.9 so I can test any of those if you don't have them. I can also test Clang eventually.
> >
> > Please update the description of the patch, since the old description says you don't open these tools from the Types dialog.
Yes, I can confirm that the loading tools if fine. It is working fine even in tha case when a single file contains multiple tools.
Please test compilation with the older versions of GCC and clang and tell me what to change.
- Aniket
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118942/#review62665
-----------------------------------------------------------
On July 19, 2014, 4:52 p.m., Aniket Anvit wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118942/
> -----------------------------------------------------------
>
> (Updated July 19, 2014, 4:52 p.m.)
>
>
> Review request for KDE Edu and David Narváez.
>
>
> Repository: kig
>
>
> Description
> -------
>
> This patch adds support for Geogebra's tool files ( .ggt files ) in kig.
>
>
> Diffs
> -----
>
> CMakeLists.txt 4b2d286
> filters/geogebra-filter.h 1b2b416
> filters/geogebra-filter.cpp 53c9253
> filters/geogebra.qrc bfe493a
> filters/geogebra.xsl a0809da
> geogebra/geogebra-transformer.h PRE-CREATION
> geogebra/geogebra-transformer.cpp PRE-CREATION
> geogebra/geogebra.qrc PRE-CREATION
> geogebra/geogebra.xsl PRE-CREATION
> modes/typesdialog.h ae245e2
> modes/typesdialog.cpp d14d0fe
>
> Diff: https://git.reviewboard.kde.org/r/118942/diff/
>
>
> Testing
> -------
>
> To open a tool file, open them from the types-dialog.
> I tested with a tool which takes 2 points and gives a line , midpoint, and a circle from these two points.
> Tested with another tool which took three points and gave a circle, hyperbola and an ellipse from these points.
> Only those tools will be supported which contain only the geogebra-objects which are currently supported by kig.
>
>
> Thanks,
>
> Aniket Anvit
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20140719/cbb5b4ab/attachment.html>
More information about the kde-edu
mailing list