search

lucastle6969 / comp371

repository for comp371 final project Fall 2017 Concordia
0 stars 0 forks source link

support for colon (OUTLINE_FONT) #106

Closed lucastle6969 closed 6 years ago

lucastle6969 commented 6 years ago

support for colon (OUTLINE_FONT) only

benwiley4000 commented 6 years ago

Wouldn't it be more logical to use '\n' than '}' for newline?

lucastle6969 commented 6 years ago

I originally tried to make it \n but I ran into some trouble because \n is already an escape character . . . I dodn't spend much time trying to figure it out , because of time constraints, but basically you can't write case '\': without some special escape key first which I didn't bother finding

benwiley4000 commented 6 years ago

You can do:

switch (char) {
    case '\n':
        // do something
    // ...
}

I just tried it.

benwiley4000 commented 6 years ago

Hey @lucastle6969 do you plan to merge this?

lucastle6969 commented 6 years ago

I haven't found a fix for the space . . you can merge if you like

benwiley4000 commented 6 years ago

Merged. I'll make a new issue for dash (negative sign) support which is the reason for the space issue I believe.