bugGNU nano - Bugs: bug #61381, the SQL syntax contains nonsense

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #61381: the SQL syntax contains nonsense

Submitter:  Benno Schulenberg <bens>
Submitted:  Mon 25 Oct 2021 12:39:03 PM UTC
   
 
Severity:  4 - Important Status:  Fixed
Assigned to:  bens Open/Closed:  Closed

Tue 30 Nov 2021 04:26:34 PM UTC, comment #1: 

Fixed in git, commit f2c9aed1, by dropping all the stuff copied from the ruby syntax.

Benno Schulenberg <bens>
Group administrator
Mon 25 Oct 2021 12:39:03 PM UTC, original submission:  

See the rules for Structure and Control flow in syntax/sql.nanorc...  The SQL language does not know things like PRIVATE or PUBLIC or RAISE or EXCEPTION.  Also, see that the second rule for structure lacks a \ before the > at the end, meaning that the regex doesn't even work as intended.  :|

Then see the rules for Strings.  The first one was taken from the asm or awk syntax, but SQL does not know "#include <file>" statements.  And the double-quoted and single-quoted rules were copied from the ruby syntax, but SQL does not know constructs like %Q{something}...  The same for shell command expansion and regular expressions -- simply copied from the ruby syntax.  And the rule for trailing whitespace comes far too early.

Arrrgh...  What a mess!  I should have posted the file to the mailing list to ask for comments before maybe accepting it, because I knew and know nothing about SQL.

Benno Schulenberg <bens>
Group administrator

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by bens (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2021-12-16 bens Open/ClosedOpen Closed
    2021-11-30 bens StatusNone Fixed
    2021-10-26 bens Summarythe SQL syntax contains crap the SQL syntax contains nonsense

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code