Go To Definition target file is incorrect

[F1F88]

Basic informations

• OS: [Windows]
• VSCode version: Latest
• Extension version: Latest

Further Information

In "include/log4sp.inc", when the LOG4SP_NO_EXT macro is not defined, header files in "include/log4sp" folder will be included.

Conversely, if the LOG4SP_NO_EXT macro is defined, header files in "include/log4sp_no_ext" folder will be included.

Therefore, it should be possible to include or jump to header files different folders depending on whether the macro LOG4SP_NO_EXT is defined.

But the actual situation is that regardless of whether the LOG4SP_NO_EXT macro is defined, Go To Logger will jump to "include/log4sp_no_ext/logger.inc"

Even without include "include/log4sp/common.inc" and "include/log4sp_no_ext/common.inc", but after deleting these two files, the error will disappear.

To Reproduce

• test_log4sp.sp

  #include <log4sp>
  
  public void OnPluginStart()
  {
      // Go to Logger jumps to "include/log4sp_no_ext/logger.inc"
      // Not "include/log4sp/logger.inc"
      Logger log = new Logger("name");
  
      // Error: no method `GetLevel` on type `Logger`
      // But it compiles fine.
      log.GetLevel();
  }

• include

  • log4sp.inc

    #if defined _log_for_sourcepawn_included
    #endinput
    #endif
    #define _log_for_sourcepawn_included
    
    #if !defined LOG4SP_NO_EXT
    
    #include <log4sp/logger>
    
    #else
    
    #include <log4sp_no_ext/logger>
    
    #endif

  • log4sp

    • common.inc

      #if defined _log_for_sourcepawn_common_included
      #endinput
      #endif
      #define _log_for_sourcepawn_common_included

    • logger.inc

      #if defined _log_for_sourcepawn_logger_included
      #endinput
      #endif
      #define _log_for_sourcepawn_logger_included
      
      methodmap Logger < Handle
      {
          public native Logger(const char[] name);
      
          public native int GetLevel();
      }

  • log4sp_no_ext

    • common.inc

      #if defined _log_for_sourcepawn_common_included
      #endinput
      #endif
      #define _log_for_sourcepawn_common_included

    • logger.inc

      #if defined _log_for_sourcepawn_logger_included
      #endinput
      #endif
      #define _log_for_sourcepawn_logger_included
      
      methodmap Logger < Handle
      {
          public Logger(const char[] name) {
              return view_as<Logger>(INVALID_HANDLE);
          }
      
          // public int GetLevel() {
          //     return 2;
          // }
      }

Code to reproduce the behaviour

Go To Definition incorrect

No method GetLevel on type Logger

Expected behaviour

If LOG4SP_NO_EXT is not defined, should jump to the include files of "include/log4sp/**".