[geany/geany] ctags unable to find variables that contain hyphen (#2679)

List overview All Threads

newer

older

[geany/geany] Blue snippets...

[geany/geany] jump to def (#2680)

Saibal

30 Nov 2020 30 Nov '20

11:15 a.m.

I have ctags file in pipe format. Properties with hyphen in name (flex-direction) does not show suggestion of possible values.

can i solve it?

-- 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/issues/2679

Attachments:

attachment.htm (text/html — 1.1 KB)

Show replies by date

elextr

30 Nov 30 Nov

11:23 a.m.

Might be difficult, a lot of Geany treats all languages as C, which doesn't allow hyphen in names.

-- 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/issues/2679#issuecomment-735695464

Saibal

11:26 a.m.

Ok thanks, I'm not very happy about this.

Of course for a web editor (css is main language) is very strange can't use this feature

-- 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/issues/2679#issuecomment-735697311

elextr

11:30 a.m.

...

Of course for a web editor (css is main language) is very strange can't use this feature

Geany isn't specifically a web editor, it does what people contribute, and mostly the C folks contributed autocomplete and other functions. If other language folks want to improve it, pull requests are welcome.

-- 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/issues/2679#issuecomment-735699647

Matthew Brush

1 Dec 1 Dec

2:01 a.m.

It works for me, though not for `flex-direction` specifically, since I tested using the tags file linked from #572 and neither one includes `flex-direction`. If I type `flex-`, it gives auto-complete list for `flex-align`, `flex-flow`, `flex-line-pack`, etc.

Tested with 1.38.

What tags file are you using?

-- 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/issues/2679#issuecomment-736148209

Saibal

8:39 a.m.

geany 1.38? last version is 1.37.1

this is an example of my ctags... it only works for tags without hyphen:

cursor -> OK border-bottom-style -> KO

# format=pipe #CSS properties pretend to be methods to provide code hint align-content||: | border-bottom-style||: none / hidden / dotted / dashed / solid / double / groove / ridge / inset / outset| border-bottom-width||: | cursor||: auto / crosshair / default / hand / move / e-resize / ne-resize / nw-resize / n-resize / se-resize / sw-resize / s-resize / w-resize / text / wait / help| direction||: | display-inside||: | display-list||: | display-outside||: | display||: none / inline / block / list-item / inline-block / table / inline-table / table-caption / table-column / table-colgroup / table-header-group / table-row-group / table-footer-group / table-row / table-cell / flex / inline-flex / grid / inline-grid / run-in|

-- 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/issues/2679#issuecomment-736284436

elextr

10:15 a.m.

@saiballo your file works for me for both hyphenated and unhyphenated tags.

While editing `geany.css` and with your tags manually loaded as `foo.css.tags` it appears to be working to me.

If I type "curs" autocomplete offers "cursor", if I type "bord" autocomplete offers "border-bottom-style" and "border-bottom-width".

Please provide how to reproduce in more detail.

Geany 1.38 is the git development version, its has no fixes in the tags area (yet).

-- 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/issues/2679#issuecomment-736334008

Saibal

10:21 a.m.

autocomplete works for me too but not for possible values. take a look on screenshots

![Schermata a 2020-12-01 10-19-04](https://user-images.githubusercontent.com/3473910/100720902-f6a8ea00-33be-11...) ![Schermata a 2020-12-01 10-17-58](https://user-images.githubusercontent.com/3473910/100720904-f7418080-33be-11...)

-- 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/issues/2679#issuecomment-736337716

elextr

10:43 a.m.

Thats a calltip, not autocompletion. Calltips require searching backward from the `(` to find a name to lookup, and thats one of the places I alluded to where only looking for C names is hard coded see the NULL at the end [here](https://github.com/geany/geany/blob/d2740f21feb84b6f951398c6f683674b2b56bfed...)

-- 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/issues/2679#issuecomment-736358704

Saibal

10:49 a.m.

ok, I'm sorry for mistake. so, no solution for that, right?

-- 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/issues/2679#issuecomment-736362201

elextr

10:55 a.m.

Not without somebody changing the code to pass the wordchars value from the filetype file. Pull requests are welcome.

-- 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/issues/2679#issuecomment-736365246

Saibal

10:58 a.m.

thanks but I'm not a C developer, so I will wait if someone can solve this (in my opinion) bug.

-- 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/issues/2679#issuecomment-736367044

1500

Age (days ago)

1501

Last active (days ago)

github-comments@lists.geany.org

11 comments

3 participants

tags (0)

participants (3)

elextr
Matthew Brush
Saibal