Skip to content

Conversation

@bberkhin
Copy link

@bberkhin bberkhin commented Jan 6, 2022

function strncasecmp does not work correctly, so unit tests fail

strncasecmp does not work correctly, so unit tests fail
formatting fix
Added the FluidConsole project. The project is a win32 console application that allows you to debug and emulate Fluidnc code on Windows.
@atlaste
Copy link
Collaborator

atlaste commented Jan 7, 2022

Whoa you just saved me a lot of work, because this was exactly the intention of how I started this x86 support branch (and I didn't knew that you can use conio like this)! I'm actually surprised that it works with candle with just these changes.

I did an extensive review, and nagged about the things that I could find. One thing that is a bit of an issue, is that you targetted the 'main' branch instead of the 'devt' branch in the PR (which means we cannot take it as-is without breaking git), but tbh I don't think it will be a lot of trouble cherry-picking the code and re-target it.

I would very much like to have these remaining things fixed so we can get this in the main branch. If you have any questions or remarks, feel free to do so, or just join us on Discord.

@bberkhin
Copy link
Author

bberkhin commented Jan 9, 2022

great that you liked my work )
I will re-target all changes to devt branch today or tomorrow and will send a new pull requsts.
Coud you send me an invitation do Discord. My id: "Boris Berkhin#3986"

@readeral
Copy link

readeral commented Jan 9, 2022

@bberkhin https://discord.gg/aKAT4Dfb

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants