natlink icon indicating copy to clipboard operation
natlink copied to clipboard

code cleanup - redefinition of UNICODE

Open dougransom opened this issue 3 years ago • 6 comments

natlinksource has preprocessor definition of UNICODE that is required, and seems to conflict with visual studio.

natlink is never going back to a non-unicode compile. the preprocesser defiintion of UNICODE should be removed out of visual studio and the conditional compiled code not required removed from natlinksource.

dougransom avatar Aug 01 '21 01:08 dougransom

Yes if I remember correctly that convention is only needed for version less than 13 of DNS.

LexiconCode avatar Aug 01 '21 02:08 LexiconCode

I think that perhaps the UNICODE preprocessor command should be renamedif it is to be kept and previous versions of Dragon supported? There is something in Visual Studio includes also called UNICODEthat we are redefining.

dougransom avatar Aug 05 '21 14:08 dougransom

Yes I could see that logic to have it renamed. It would be good to have previous 13/14 versions supported as the upgrade cost is now 250$. However I know your time/energy is limited so I wouldn't worry about that now. Let me know what you rename it too and how to enable it which might give me a chance to poke and prod.

LexiconCode avatar Aug 05 '21 14:08 LexiconCode

I'm not going to rename it, just highlighting the issue for the next person who pokes at the code.

dougransom avatar Aug 05 '21 14:08 dougransom

Fusentasticus, do you have an opinion about the "#ifdef UNICODE" lines in the .cpp code?

quintijn avatar Feb 16 '22 15:02 quintijn

There is some weirdness around unicode, and there were some changes to do with playstring or something that required unicode conversions and using the non-unicode calls to natlink. Maybe lets just leave this haning around for a while.

dougransom avatar Nov 30 '22 01:11 dougransom

Let us leave this for now...

quintijn avatar Jun 06 '24 09:06 quintijn