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
The mil_misc_tools.FprintFactory factory class currently uses a custom class to hold the unicode characters to color text as individual class variables. To obtain these unicode characters from outside the module, other modules will frequently refer to the color as a string.
Instead, this could be converted to an enumerator to provide greater code clarity and reliability, while also adding some code completion features. This would help prevent an unassuming MIL member from accidentally trying to print a message in a color that doesn't exist to FprintFactory!
For example, this is how the code is structured now:
The
mil_misc_tools.FprintFactory
factory class currently uses a custom class to hold the unicode characters to color text as individual class variables. To obtain these unicode characters from outside the module, other modules will frequently refer to the color as a string.Instead, this could be converted to an enumerator to provide greater code clarity and reliability, while also adding some code completion features. This would help prevent an unassuming MIL member from accidentally trying to print a message in a color that doesn't exist to
FprintFactory
!For example, this is how the code is structured now:
This could be restructured as:
The text was updated successfully, but these errors were encountered: