Skip to content
This repository was archived by the owner on Nov 5, 2020. It is now read-only.

Comments

G92 4-axis fix and Mac/Linux build error#1

Merged
LETARTARE merged 2 commits intoLETARTARE:Letartarefrom
srichy:Letartare
Nov 3, 2014
Merged

G92 4-axis fix and Mac/Linux build error#1
LETARTARE merged 2 commits intoLETARTARE:Letartarefrom
srichy:Letartare

Conversation

@srichy
Copy link

@srichy srichy commented Nov 3, 2014

  1. Replaced backslash with POSIX-style slash for directory separator in include.

  2. G92 T axis value missing offset in 4-axis mode in GCV

    When clicking on the zero-axes button (to the right of the Home
    button), a G92 command of the form

    G92 x0 y0 z0 a

    would be incorrectly emitted. This change moves the toLower() method
    so that it is called on the QString instance of the axis name
    attribute directly rather than on the result of the append() method.

When clicking on the zero-axes button (to the right of the Home
button), a G92 command of the form

  G92 x0 y0 z0 a

would be incorrectly emitted.  This change moves the toLower() method
so that it is called on the QString instance of the axis name
attribute directly rather than on the result of the append() method.
Replaced backslash with POSIX-style slash for directory separator in
include.
@LETARTARE LETARTARE self-assigned this Nov 3, 2014
LETARTARE added a commit that referenced this pull request Nov 3, 2014
G92 4-axis fix and Mac/Linux build error
@LETARTARE LETARTARE merged commit d19cf00 into LETARTARE:Letartare Nov 3, 2014
@LETARTARE
Copy link
Owner

@srichy
can you make a compilation for Linux?
Best regards

@srichy
Copy link
Author

srichy commented Nov 17, 2014

I've committed a Linux fix to my repo and will create a new pull request for that.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants