@dolik-rce commented on this pull request.


> @@ -4,7 +4,7 @@
 # https://kotlinlang.org/docs/reference/keyword-reference.html
 primary=abstract assert break case catch class const continue default do else 
enum extends final finally for goto if implements import instanceof interface 
native new package private protected public return static strictfp super switch 
synchronized this throw throws transient try volatile while true false null as 
fun in object typealias val var when by constructor delegate dynamic field file 
get init param property receiver set setparam where actual annotation companion 
const crossinline data expect external infix inline inner internal lateinit 
noinline open operator out reified sealed suspend tailrec vararg field it
 # https://kotlinlang.org/docs/reference/basic-types.html
-secondary=Double Float Long Int Short Byte NaN Void
+secondary=Boolean Byte Char Double Float Long Int Short String UByte UInt 
ULong UShort Void

I'd maybe ditch `Void` too, it's only used in Java IIRC.

What about `Nothing` and `Unit`? Those are kind of basic types too, aren't 
they? But the link above doesn't mention them, so it is kind of gray area.

Otherwise it looks fine to me (as a Kotlin user).

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3456#pullrequestreview-1380585736
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/3456/review/[email protected]>

Reply via email to