atom / language-sql

SQL package for Atom
Other
38 stars 50 forks source link

language-sql wrongly interprets backslash as escaping char for varchars. #61

Closed yaras closed 5 years ago

yaras commented 7 years ago

Prerequisites

Description

language-sql wrongly interprets backslash as escaping char for varchars.

Steps to Reproduce

  1. Add content:

    begin
    dbms_output.put_line('First line');
    
    dbms_output.put_line('\');
    
    dbms_output.put_line('Second line');
    end;
  2. Change syntax to SQL

Expected behavior:

Line dbms_output.put_line('Second line'); should be formatted exactly as dbms_output.put_line('First line');

Actual behavior:

language-sql

Reproduces how often:

100%

Versions

Atom : 1.16.0 Electron: 1.3.13 Chrome : 52.0.2743.82 Node : 6.5.0 Windows: 10 language-sql: 0.25.4

aawadall commented 6 years ago

Is there a solution for the above issue?

dorian-marchal commented 5 years ago

same issue

Akaizoku commented 5 years ago

Duplicate of #13

Still has not been fixed since May 2015.

Arcanemagus commented 5 years ago

Thanks for pointing that out @Akaizoku 😉.

We noticed that this is a duplicate of https://github.com/atom/language-sql/issues/13. You may want to subscribe there for updates.

Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.

For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.