[vim/vim] duplicate runtime/syntax vimscript for Oracle PL/SQL (Issue #15310)

Yinzuo Jiang

unread,

Jul 20, 2024, 1:42:36 AM7/20/24

to vim/vim, Subscribed

I found that runtime/syntax/plsql.vim and runtime/sqloracle.vim are both for Oracle Procedural SQL (PL/SQL) language.

Should we keep just one file for PL/SQL? @lee-lindley @chrisbra

—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.

Christian Brabandt

unread,

Jul 20, 2024, 6:16:05 AM7/20/24

to vim/vim, Subscribed

It's been a while, since I had to be working with PL/SQL and SQL Oracle scripts and there is some overlap.

Hm, no I believe sqloracle is used mostly for SQL statements (that may of course also contain plsql syntax), while plsql is more for scripts containing only plsql functions and definitions.

Nowadays, I don't work with Oracle anymore, so I cannot test it, but I think it is fine to keep both versions.

—
Reply to this email directly, view it on GitHub.

You are receiving this because you are subscribed to this thread.

Lee Lindley

unread,

Jul 20, 2024, 7:13:41 AM7/20/24

to vim/vim, Subscribed

plsql.vim includes Oracle SQL syntax as well as PL/SQL and can be used for regular SQL. It is more comprehensive on Oracle keyword identification including sqlplus keywords, and has syntax highlighting enhancementss for quoted strings and double quoted identifiers. It also includes code folding while sqloracle.vim does not.

sqloracle.vim behaves differently, neither better nor worse. If one uses SQL in multiple database types along with Oracle, it may be a better choice. It was also the default choice for a very long time. Many may prefer it.

—
Reply to this email directly, view it on GitHub.

You are receiving this because you are subscribed to this thread.

Christian Brabandt

unread,

Jul 20, 2024, 7:32:27 AM7/20/24

to vim/vim, Subscribed

Thanks. closing then.

—
Reply to this email directly, view it on GitHub.

You are receiving this because you are subscribed to this thread.

Christian Brabandt

unread,

Jul 20, 2024, 7:32:30 AM7/20/24

to vim/vim, Subscribed

Closed #15310 as not planned.

—
Reply to this email directly, view it on GitHub.

You are receiving this because you are subscribed to this thread.

Reply all

Reply to author

Forward