Skip to content

Improve switch header/source matching #2635

Closed

Description

From #618 (comment)

I have similar issue. I have the following folder structure (simplified)

componentA/tests/inc/TestDataBuilder.h
componentA/tests/src/TestDataBuilder.cpp
componentB/tests/inc/TestDataBuilder.h
componentB/tests/src/TestDataBuilder.cpp

When componentB/tests/inc/TestDataBuilder.h is opened and I press Alt+O, the editor jumps to componentA/tests/src/TestDataBuilder.cpp instead of the source file in componentB. When I press Alt+O a second time, it jumps to componentA/tests/inc/TestDataBuilder.h. It seems that the algorithm simply picks the first header it finds in alphabetical order without considering locality.

Activity

  1. AHKol commented on Oct 28, 2018

    @AHKol

    I'm new to the vscode-cpptools repo. Since this looks like a good first bug I will give it a shot.

    I managed to replicate it already.

  2. bobbrow commented on Oct 29, 2018

    @bobbrow
    Contributor

    AHKol, we appreciate your willingness to help out! Unfortunately this is something that is implemented in our language server which is not open source at this time. The bugs that are fixable by the community will be marked with the "help wanted" label.

  3. alanbirtles commented on May 8, 2019

    @alanbirtles

    I'm having similar issues, in my case I have headers with generic names like "Connection.h" "Object.h", when I switch from the cpp file to the header vscode switches to headers with the same name in system libraries (e.g. GCC in the case of Object.h and AWS SDK for Connection.h).
    If this issue can't be fully fixed by finding the "nearest" matching file can it at least search workspace files before library files?

  4. rhaschke commented on Nov 3, 2019

    @rhaschke

    I have similar issues: Having multiple matching header files found in c_cpp_properties.json's includePath, it seems to pick the last match (instead of the first one).

  5. kisielk commented on Dec 4, 2019

    @kisielk

    I'm having trouble getting the switch to work at all, even with filenames like {Src,Inc}/module.{cpp,h}

  6. SteelBlueVision commented on Feb 16, 2021

    @SteelBlueVision

    This is absolutely insane, you can't use Alt-O (shortcut for C/C++: Switch Header/Source) to switch from header to source, if the two lie in different directories. This is true even if the directory where source files reside is mentioned in the /configurations/browse/path JSON array in c_cpp_properties.json as (for example) "/complete/absolute/path/to/source_files/*" . It should be noted that all of the source files lie in the aforementioned dir - there are no nested subdirectories in .../source_files, hence the single asterisk was used to not perform a recursive search (not that that would have helped anyways - I've tried it out of desperation).

    It's so broken that it can't even go from header to source with Alt-O, if the source is already loaded and present in a tab right next to it in the editor, and these are the only two files that are currently open! It has no trouble going from source to header, though, so clearly it seems some sort of connection between the two.

    Note: I am using a .hpp extension for headers and .cpp for source files, if that is helpful.

  7. sean-mcmanus commented on Feb 17, 2021

    @sean-mcmanus
    ContributorAuthor

    mgoldshteyn This issue is tracking some known limitations with switch header/source in which it'll pick a target with matching name at the unwanted folder, but it sounds like you may be hitting a different issue in which no target is found. Are you able to provide a simple repro? I'm not reproing the issue with the simple repro in this screenshot:

    image

  8. 7 remaining items

  9. sean-mcmanus commented on Mar 8, 2023

    @sean-mcmanus
    ContributorAuthor

    @sweihub There's not currently a way to prioritize an include directory, but you can exclude one via C_Cpp.files.exclude (assuming you don't mind losing global symbols in those files).

  10. mua commented on Sep 28, 2023

    @mua

    This is one of those things breaks entire keyboard workflow frequently.

  11. StefanoPierini commented on Feb 9, 2024

    @StefanoPierini

    Meanwhile, the workaround I found to this issue is to install the extension c-mantic and bind the shortcut to its "Switch Header Source". It seems to work correctly.

  12. moved this to Pull Request in cpptoolson Jan 4, 2025
  13. moved this from Pull Request to Done in cpptoolson Feb 4, 2025
  14. added
    fixedCheck the Milestone for the release in which the fix is or will be available.
    enhancementImprovement to an existing feature
    and removed on Feb 7, 2025
  15. sean-mcmanus commented on Feb 12, 2025

    @sean-mcmanus
    ContributorAuthor

    This is fixed with https://github.com/microsoft/vscode-cpptools/releases/tag/v1.24.0 . If there are additional scenarios that are not fixed still we may open a new issue to track that additional improvement.

  16. locked and limited conversation to collaborators on May 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Type

Projects

  • Status
    Done

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions