search

EliotVU / UnrealScript-Language-Service

Bringing a work-in-progress intelliSense to ye olde UnrealScript :)
MIT License
48 stars 9 forks source link

'intrinsic' is not recognized as a field modifier #125

Closed PolaricEntropy closed 2 years ago

PolaricEntropy commented 3 years ago

The 'intrinsic' keyword is not recognized as a field modifier. It works fine to use 'intrinsic' instead of 'native' for classes though.

https://puu.sh/GINmQ/2b417829ba.png

midnight-dev commented 3 years ago

Weird. It can be found in several places already, like UnrealScript syntax YAML, L183 and UC Lexer L124.

Much of my recent shenanigans with UScript just involve mutators for UT2004, so intrinsic definitions are a bit out of my scope. Does that compile without issue, even if the VSCode extension complains?

PolaricEntropy commented 3 years ago

Yes it does, and the line doesn't get an error by the extension if I use the "native" keyword as a field modifier. I conferred with my local UE1 expert, hanfling, and he told me that "intrinsic" and "native" are mutually interchangeable in all places. According to the unreal wiki "intrinsic" is a valid keyword for UE1 and UE2, but that page might not be entirely accurate since it's gotten a few other things wrong. It might still be a good general guideline.

EliotVU commented 2 years ago

Sorry for the late response, but thanks for submitting this issue! It seems like I may have missed this keyword for variables. I have a pending fix for this in the develop branch, the fix should go live with version 0.5.

PolaricEntropy commented 2 years ago

Is v0.5 slated for release soon?

EliotVU commented 2 years ago

I plan to publish it in a moment, that is if the publishing features are still working on my end 📦