Andy,

When originally written, it was basically a cut and paste from function
emc_rel_cmd_pos (etc. for the other functions) found in emcsh.cc, mainly
with the tcl stuff removed. I do not recognize that code, unless they
diverged at some point. We should check the diffs to see either when this
was introduced or when they diverged.

Regards,
Eric


A couple of people have reported puzzlement with linuxcncrsh and the
reporting of REL_CMD_POS Looking at the code, there does seem to be what is
probably a long-lasting cut-and-paste error.

in emcrsh.cc in getRelCmdPos (1644) and getRelActPos (1704) and in half of
getPosOffset (1765) the g5x offsets are added or subtracted twice.

I accept that this _might_ be to avoid a multiplication in time-critical
code. But I doubt it.

I have a strong suspicion that the second g5x offset was meant to be a
g92 offset, as is the case in the rotary axis calcs in getPosOffset.



------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Emc-developers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/emc-developers

Reply via email to