Hello!
While chasing down these `clang` warnings I found that some code is redundant
and can be safely removed:
```
main/parse.c:5007:26: warning: unused function 'FallbackParser'
[-Wunused-function]
static parserDefinition *FallbackParser (void)
^
main/parse.c:5040:26: warning: unused function 'CTagsParser'
[-Wunused-function]
static parserDefinition *CTagsParser (void)
^
main/parse.c:5355:26: warning: unused function 'CTagsSelfTestParser'
[-Wunused-function]
static parserDefinition *CTagsSelfTestParser (void)
^
```
This is because we explicitly define `EXTERNAL_PARSER_LIST ` in tm_parsers.h at
https://github.com/geany/geany/blob/master/src/tagmanager/tm_parsers.h#L16 so
guess no need to check for not defined case?
Thanks!
You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/2890
-- Commit Summary --
* Remove redundant code
-- File Changes --
M ctags/main/parse.c (338)
M ctags/main/parse_p.h (13)
-- Patch Links --
https://github.com/geany/geany/pull/2890.patch
https://github.com/geany/geany/pull/2890.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2890