Support Dynamic SQL syntax

[yedane]

Type: LanguageService

Describe the bug

• OS and Version: Win 10
• VS Code Version: 1.34
• C/C++ Extension Version: 0.23.1
• Other extensions you installed (and if the issue persists after disabling them):

We have to use dynamic SQL statements, and in the synthax we have to put a ":" before a variable, but it seems that break the IntelliSense.
Please have a look on the documentation below: Oracle dynamic statements

Actually when we want to drive into dba_test_with_proc_and_without_colon function it show us the header instead of the function declaration.
But when we drive into dba_test_with_proc_and_without_colon it shows us the declaration.

To Reproduce

1. Go to the main file
2. Go to Declaration (F12) on the dba_test_with_proc_and_colon, it drives to the header declaration
3. Go to Declaration (F12) on the dba_test_without_proc, it drives to the function declaration
4. Go to Declaration (F12) on the dba_test_with_proc_and_without_colon, it drives to the function declaration

Expected behavior

when we go to Declaration (F12) on the dba_test_with_proc_and_colon, it must drive to the function declaration instead of the header

Code source

main.c

#include "hello.h"

int main()
{
  int vl_err = 0;
  short vl_num = 0;

  vl_err = dba_test_with_proc_and_colon(vl_num);
  vl_err = dba_test_without_proc(vl_num);
  vl_err = dba_test_with_proc_and_without_colon(vl_num);
  return 0;
}

hello.h

int dba_test_with_proc_and_without_colon(short va_num);
int dba_test_with_proc_and_colon(short va_num);
int dba_test_without_proc(short va_num);

hello.c

int dba_test_without_proc(short va_num)
{
  int vl_ret_cod = 0;

  if (va_num > 1)
    vl_ret_cod = 1;
  return vl_ret_cod;
}

int dba_test_with_proc_and_without_colon(short va_num)
{
  int vl_ret_cod = 0;

  EXEC SQL BEGIN DECLARE SECTION;
  char vl_string[2048] = {0};
  EXEC SQL END DECLARE SECTION;
  strcpy(vl_string, "select * from tab");
  EXEC SQL PREPARE select from vl_string;
  return vl_ret_cod;
}

int dba_test_with_proc_and_colon(short va_num)
{
  int vl_ret_cod = 0;

  EXEC SQL BEGIN DECLARE SECTION;
  char vl_string[2048] = {0};
  EXEC SQL END DECLARE SECTION;

  strcpy(vl_string, "select * from tab");
  EXEC SQL PREPARE select from :vl_string;
  return vl_ret_cod;
}

In the zip, you can find the source with we repro the issue
Intellisense.zip

[Colengms]

Hi @yedane,

I don't think I have enough information to repro your issue.

The 'main.c' and 'Header file' you provided appear to be the same file. Did you intend to send another file? Perhaps one which declared the macros you are using?

Consider creating a repo with a self-contained example of the issue, which we could pull to investigate.

Thanks,

• Colen

[yedane]

Hello @Colengms,

My bad, I edited my post

[Colengms]

Dynamic SQL is not standard C++. Is it implemented using macros, a special compiler, or a preprocessor?

We don't support this syntax. We could consider this a feature request..

[yedane]

It's preprocessor.

Is it possible to ignore it during intellisense parsing ?

[sean-mcmanus]

@yedane You could surround the SQL code in

#ifndef __INTELLISENSE__
#endif

...but I don't think there's any way for us to auto-detect this.

[Colengms]

Hi @yedane,

I think if the Dynamic SQL were C++ preprocessor based, squiggles would be correct. IntelliSense parsing should consider macro contents. However, it's unclear to me what could be within a preceding macro that would make a ":" valid before a variable name. My suspicion is that it's not preprocessor based.

[yedane]

@Colengms

Dynamic SQL is implemented by using PRO C, it's finally a precompiler that transform into C code: PRO C More information

Here is an code example before and after compilation: Simple example

[Colengms]

Thanks @yedane. Since the syntax is processed by an alternate (pre)compiler and not C++ preprocessor macros, that would explain the squiggle. Let's keep this issue open as a feature request to support (or at least not squiggle) Dynamic SQL syntax.