-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
What is missing from the current name printing? #25
Comments
LOL I was just trying to open this exact same ticket 😆 It's funny, it manages to not shadow just one variable, and everytihng else is shadowed. oops. |
Proposal @jonsterling @ecavallo
|
@favonia I think Evan and I agree this needs to be fixed, but don't care at all in what manner it is fixed 😆 |
@jonsterling For now, I have to simplify the scheme to:
because I would like to use a proper UTF-8 library to strip off the suffix. (I wonder if @freebroccolo knows an immortal choice of UTF-8 libraries.) |
@favonia We are already using an immortal utf-8 library (uuseg, uuseg_string)... But please, can we just do something simple? Even without the subscripts if it is easier. This (technically minor) issue is such that any fix at all would lead to a great improvement, so we really don't need to boil the ocean for it 😆 |
@jonsterling Another subtlety is that we should also avoid shadowing names in the global signature (names of datatypes, constructors, etc). |
@favonia Yes, agreed!! |
I agree with Jon that the current choice of libraries for unicode stuff is probably the best available. Regarding pretty printing with the subscripts like this, I implemented something similar here. That algorithm cycles through the full alphabet ('a' - 'z') for each new subscript but it should be easy to change to just use one letter. |
x
,y
,z
,w
for expression variables andi
,j
,k
for dimension variables.The text was updated successfully, but these errors were encountered: