Skip to content

add kotlin parser from uctags#2

Open
dolik-rce wants to merge 1 commit intotechee:ctags_sync5from
dolik-rce:kotlin-parser
Open

add kotlin parser from uctags#2
dolik-rce wants to merge 1 commit intotechee:ctags_sync5from
dolik-rce:kotlin-parser

Conversation

@dolik-rce
Copy link

@dolik-rce dolik-rce commented Dec 28, 2020

Hello,

As a long time happy Geany user here, I've been missing better Kotlin support last few years. I have already added a basic Kotlin parser in to uctags (see universal-ctags/ctags#2769) and this is my attempt to make use of it in Geany.

I had to use the latest ctags_sync branch from @techee, because the parser uses some features of uctags that are not currently present in master. Unfortunately, I couldn't figure out how to create this PR directly in https://github.com/geany/geany ☹️ So I'll create it here for the time being (that is probably until ctags_sync5 branch is not merged) and create and additional issue in the main Geany project to draw attention of appropriate maintainers to it.

@@ -0,0 +1,171 @@
/*
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not really sure about the naming here. I see all of the files currently have geany_ prefix, but this one is directly copied from ctags repo, so it doesn't seem to be "geany specific" enough to warrant a prefix. If I understand this incorrectly, please let me know and I'll rename it.

techee pushed a commit that referenced this pull request Nov 25, 2021
Also store some UI-related prefs in the new session.conf, that is:

treeview_position=156
msgwindow_position=593
geometry=0;57;1176;825;0;
techee pushed a commit that referenced this pull request Nov 25, 2021
By Thomas Martitz (2) and Adam Morris (1)
* upstream/session_split:
  session.conf split follow-up #2 (geany#2779)
  session.conf split follow-up
  Split geany.conf into geany.conf (preferences) and session.conf (recent files and VTE session) (geany#2776)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant