Generated insight for jmp_buf global variable declarations.

[loonatick-src]

When including setjmp.h and declaring a global variable with type jmp_buf, the generated insight is just std::jmp_buf = std::jmp_buf.

Example source code (line 3):

#include <cstdio>
#include <csetjmp>
std::jmp_buf jump_buffer;

void foo() {
    printf("foo: Entered\n");
    longjmp(jump_buffer, 1);  // Jump back to where setjmp was called
    printf("foo: This won't be executed\n");
}

int main() {
    if (setjmp(jump_buffer) == 0) {
        printf("main: Calling foo\n");
        foo();
    } else {
        printf("main: Jumped back from foo\n");
    }

    printf("main: Exiting\n");
    return 0;
}

Generated insight (line 3):

#include <cstdio>
#include <csetjmp>
std::jmp_buf = std::jmp_buf();


void foo()
{
  printf("foo: Entered\n");
  longjmp(jump_buffer, 1);
  printf("foo: This won't be executed\n");
}


int main()
{
  if(_setjmp(jump_buffer) == 0) {
    printf("main: Calling foo\n");
    foo();
  } else {
    printf("main: Jumped back from foo\n");
  } 
  
  printf("main: Exiting\n");
  return 0;
}

[andreasfertig]

Hello @loonatick-src,

thanks for reporting this nice issue. I (re)learned something today :-) std::jmp_buf is roughly defined as:

typedef int jmp_buf[30];

This makes the type behind it an array, but the type does not appear as such. The C++ Insights code did not consider that and was looking for [ to insert the variable name.

A fix will be on its way soon.

Andreas