You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I tried to update the code to support the full international morse code standard with what I thought were single characters (minus CH) to the unordered map kMorseCode in ggmorse.cpp, but I got multi-character character constant warnings when compiling and bigger problems when running. Perhaps this is why you left these out to begin with. The symbols in question, some which are shared by other non-latin symbols and pose another interesting issue, are:
I tried converting the unordered map to strings instead of characters, but it just turned into a big mess. I'm thinking that's the way it probably should be though in order to fully support all possibilities.
The text was updated successfully, but these errors were encountered:
Multi-byte characters are currently not supported by ggmorse.
I think the simplest solution that you can do is keep the internal map as it is and extend it with a few extra single-byte characters and then perform the mapping to your multi-byte characters outside of ggmorse.
Maybe a flag could be implemented that would output the morse code sequences instead of mapping them to characters. That way, a follow-on program could handle the mapping with whatever characters are needed without having to worry about updating the mapping inside of ggmorse.
I tried to update the code to support the full international morse code standard with what I thought were single characters (minus CH) to the unordered map kMorseCode in ggmorse.cpp, but I got multi-character character constant warnings when compiling and bigger problems when running. Perhaps this is why you left these out to begin with. The symbols in question, some which are shared by other non-latin symbols and pose another interesting issue, are:
I tried converting the unordered map to strings instead of characters, but it just turned into a big mess. I'm thinking that's the way it probably should be though in order to fully support all possibilities.
The text was updated successfully, but these errors were encountered: