why do not compatible FlagsForFile in .ycm_extra_conf.py

[haolongzhangm]

after update ycm repo, find need chang FlagsForFile to Settings
why do not compatible?

[puremourning]

Please see CONTRIBUTING.md. You should lot need to change anything. Please provide diagnostics.

[haolongzhangm]

@puremourning if I do not change FlagsForFile to Settings
use YcmCompleter GoToDefinition will failed: RuntimeError: Cannot jump to location

more detail:

old ycm@afd69b382844315812fd48912eaa9fa47cba3a8d
all work fine

update ycm with command:
git pull --rebase
git submodule update --init --recursive
python3 install.py --clang-completer

then same project, so many symbol will return err: : RuntimeError: Cannot jump to location

then i moidy :

diff --git a/.ycm_extra_conf.py b/.ycm_extra_conf.py
index 37f9395..bce97ec 100644
--- a/.ycm_extra_conf.py
+++ b/.ycm_extra_conf.py
@@ -112,7 +112,7 @@ def GetCompilationInfoForFile( filename ):

This is the entry point; this function is called by ycmd to produce flags for

a file.

-def FlagsForFile( filename, **kwargs ):
+def Settings( filename, **kwargs ):
if database:

will be ok

[bstaletic]

FlagsForFile has been deprecated for a long time. The new clangd completer supports only Settings. FlagsForFile became limiting for us and so Settings was made that can do a lot more.

[haolongzhangm]

the issue commit at : 123f0c0

[bstaletic]

No, it is not. That commit is very much unrelated to C++ flag handling. Clangd completer required Settings from the start.

[haolongzhangm]

@bstaletic
i build with --clang-completer not clangd

[haolongzhangm]

@bstaletic
Consider the this scenario:
a project, as history, so many Developers use old ycm, so .ycm_extra_conf.py use FlagsForFile by default, I need modify to Settings at local,

the better way is make it compatible both for compatible or Settings?

[bstaletic]

The old completer still supports FlagsForFile, but you haven't posted the information requested by the issue template.
Please see CONTRIBUTING.md, user support is not what github issue tracker is for.