distal-attribute
distal-attribute
distal-attribute
distal-attribute

Bug?

User, date Message
Written by kpeters58Money, Euro
5 months ago
Category: General
1 posts since Wed, 12 Mar 14
Hi,

the latest (8 3 0 4766) version seems to have issues with comments (--) in the query window:

If you have 2 'commands' in a query window and one of them is commented out (and the comment -- is not separated from the command by at least one space) it is shown as commented (grayed out), yet it is still being executed

Written by ansgar
4 months ago
5015 posts since Fri, 07 Apr 06
The underlying SynEdit highlighter has a small gitch here. Two dashes must be followed by a space in order to be a comment: "-- ". If there is no space, then it's not a comment, and MySQL takes it as part of the query.

SynEdit, however, displays also two dashes without the following space as comments. This is an error I've ever wanted to submit to their bugtracker. I'll see if it probably was reported before.
Written by ansgar
4 months ago
5015 posts since Fri, 07 Apr 06
Oh, I have already filed it as a bugreport, 4 years ago...

https://sourceforge.net/p/synedit/bugs/405/
Written by kalvaro
4 months ago
594 posts since Thu, 29 Nov 07
Most likely SynEdit treats SQL as a single language and that's a valid comment in all other dialects (including standard SQL).
Written by ansgar
4 months ago
5015 posts since Fri, 07 Apr 06
Perhaps I can finally fix that. The SynEdit project unfortunately does not seem very active. I wonder why that's the case, as quite a few applications use the SynEdit highlighter.
Written by ansgar
3 months ago
5015 posts since Fri, 07 Apr 06
I think I got it, in r4790.
Written by Oblepihus
3 months ago
1 posts since Fri, 20 Jun 14
Hello!

I'm using HeidiSQL with MSSQL Server and for me there could be no spaces and the string would be still commented.

Maybe it's possible to check which server we're connected to and then use different highlight?

Thank you!
Written by ansgar
3 months ago
5015 posts since Fri, 07 Apr 06
Indeed. I thought MSSQL would behave differently. I guess it's required to make a server/version conditional again here. Sigh.
 

Please login to leave a reply, or register at first.