Updated BuildGoToResponse to take an optional fourth parameter, file_only,

which indicates whether or not the client only needs to go to the file and can
ignore the line and column number. Modified the SwitchSourceHeader call to
specify True for that. Currently is the only command for the clangd completer
that makes use of it.
Also added SwitchSourceHeader item to the list in subcommands test for clangd.

Author: mibu138

ycmd/completers/cpp/clangd_completer.py

@@ -316,9 +316,14 @@ def SwitchSourceHeader( self, request_data ):
     uri = lsp.FilePathToUri( request_data[ 'filepath' ] )
     request  = lsp.BuildRequest( request_id, "textDocument/switchSourceHeader",
             { "uri": uri } )
-    response = self.GetConnection().GetResponse( request_id, request, 30 )
+    response = self.GetConnection().GetResponse( request_id, request,
+            language_server_completer.REQUEST_TIMEOUT_COMMAND )
     filepath = lsp.UriToFilePath( response[ 'result' ] )
-    return responses.BuildGoToResponse( filepath, 1, 1, "switch" )
+    # We don't have a specific location in the file we need to go to so
+    # we just arbitrarily choose line 1, column 1. The client can choose
+    # to ignore them.
+    return responses.BuildGoToResponse( filepath, 1, 1, description = filepath,
+            file_only = True )
 
 
   def ShouldCompleteIncludeStatement( self, request_data ):

ycmd/responses.py

@@ -67,19 +67,22 @@ def __init__( self ):
 
 
 # column_num is a byte offset
-def BuildGoToResponse( filepath, line_num, column_num, description = None ):
+def BuildGoToResponse( filepath, line_num, column_num, description = None,
+        file_only = False ):
   return BuildGoToResponseFromLocation(
     Location( line = line_num,
               column = column_num,
               filename = filepath ),
-    description )
+    description, file_only )
 
 
-def BuildGoToResponseFromLocation( location, description = None ):
+def BuildGoToResponseFromLocation( location, description = None,
+        file_only = False ):
   """Build a GoTo response from a responses.Location object."""
   response = BuildLocationData( location )
   if description:
     response[ 'description' ] = description
+  response[ 'file_only' ] = file_only
   return response
 
 

ycmd/tests/clangd/subcommands_test.py

@@ -594,7 +594,8 @@ def test_Subcommands_DefinedSubcommands( self, app ):
                                                'GoToSymbol',
                                                'GoToType',
                                                'RefactorRename',
-                                               'RestartServer' ] ) )
+                                               'RestartServer',
+                                               'SwitchSourceHeader' ] ) )
         },
         'route': '/defined_subcommands',
     } )