Review Request 118785: Segment Axis object in KIG ( GSoC 14 )

Maurizio Paolini paolini at dmf.unicatt.it
Wed Jun 18 11:35:56 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118785/#review60377
-----------------------------------------------------------


The old segment_axis macro required as argument a segment, whereas the new construction
depends on two points... this could lead to some confusion, and actually, the name
SegmentAxisABType is a little bit incorrect.
A user that is accustomed to the old segment_axis would find the use of the new 
SegmentAxisABType different.
Question: what is the reason to add this new construction?

- Maurizio Paolini


On June 18, 2014, 8:55 a.m., Aniket Anvit wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118785/
> -----------------------------------------------------------
> 
> (Updated June 18, 2014, 8:55 a.m.)
> 
> 
> Review request for KDE Edu and David Narváez.
> 
> 
> Repository: kig
> 
> 
> Description
> -------
> 
> This patch adds the segment axis object to KIG. This new object will take two points as input and generates a line which would perpendicularly bisect the segment constructed with those two points.
> 
> 
> Diffs
> -----
> 
>   macros/CMakeLists.txt 8abf249 
>   macros/segment_axis.kigt e86b90c 
>   misc/builtin_stuff.cc 28846c8 
>   objects/line_type.h 3ea252b 
>   objects/line_type.cc dbf575c 
> 
> Diff: https://git.reviewboard.kde.org/r/118785/diff/
> 
> 
> Testing
> -------
> 
> Took 6 points and constructed the perpendicular-bisectors choosing random pair. Tested with semgents too.
> 
> 
> Thanks,
> 
> Aniket Anvit
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20140618/61a2652b/attachment.html>


More information about the kde-edu mailing list