[IGSTK-Users] CrossHairSpatialObject

[Michel Audette]

Hi Alvaro,

I can have look at the code and get back to you today. At first glance, it
does look funny. I'll report back shortly.

Cheers,

Michel

2009/10/7 Alvaro Gómez <agomez at fing.edu.uy>

> I'm working with Release 4.2
>
> In
> void  CrossHairSpatialObject::AttemptSetCursorPositionProcessing()
>
> the cursor position is checked against the bounding box. Is this check
> correct?
> ------------------------------------------------
>  if( m_CursorPositionToBeSet[0] < bounds[0] ||
>      m_CursorPositionToBeSet[1] > bounds[1] ||
>      m_CursorPositionToBeSet[2] < bounds[2] ||
>      m_CursorPositionToBeSet[3] > bounds[3] ||
>      m_CursorPositionToBeSet[4] < bounds[4] ||
>      m_CursorPositionToBeSet[5] > bounds[5] )
>    {
>    validPosition = true;
>    }
> ------------------------------------------------
>
> m_CursorPositionToBeSet[3], m_CursorPositionToBeSet[4] and
> m_CursorPositionToBeSet[5] are not valid values.
>
> The if clause seems to check an invalid position, not a valid position.
>
>
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
>
> Follow this link to subscribe/unsubscribe:
> http://public.kitware.com/cgi-bin/mailman/listinfo/igstk-users
>



-- 
Michel Audette, Ph.D.
R & D Engineer,
Kitware Inc.,
Chapel Hill, N.C.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/igstk-users/attachments/20091007/643052ef/attachment.html>