-
Notifications
You must be signed in to change notification settings - Fork 101
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
<CODE SAS> highlighting inside PROC SQL #114
Comments
Please, post the example also as text (the picture is not selectable). |
It is a multiline comment according to definition file: Line 47 in 5861c58
According to most manuals the comment should just start with Some examples with syntax highlighter of Github, no idea if that one is right at all?: proc sql;
select count(*) AS Total, d.OwnRent, d.AssistedLiving, d.NursingHome, d.LiveWFam
from Dwelling2 as d
group by d.OwnRent, d.AssistedLiving, d.NursingHome, d.LiveWFam
order by Total DESC;
quit;
proc sql; *df
dfdf;
proc sql; *text;
proc sql; (*)text;
proc sql; *)text;
proc sql; (*text;
proc sql; k*ktext;
proc sql;*ktext;
proc sql;k* text;
proc sql; k* text;
proc sql; *text 2spaces before So question is, what should be the exact logic for these comments? |
The IDE that comes with SAS allows /* */ style commenting inside PROC SQL, |
Thanks! You have correctly articulated another highlighting inconsistency. |
I see some issues with getting this to work in GeSHi 1.0 as this requires some context we don't usually have and that also can't easily be done with constant PCRE look-behind. Thus most likely having GeSHi work out the two separate language contexts on its own might be somewhat complicated and probably not possible with the GeSHi 1.0 parser. GeSHi 1.1 should be able to pull this off though. Solutions in GeSHi 1.0 will likely be a compromise on what is reliably detectable. |
SAS has a PROC SQL statement which contains different (SQL) syntax. The syntax highlighting should reflect this.
Problem code: the asterisk in the select statement is treated as a SAS comment.
The text was updated successfully, but these errors were encountered: