search

loganch / AutoIt-VSCode

AutoIt Extension for Visual Studio Code
https://marketplace.visualstudio.com/items?itemName=Damien.autoit
MIT License
74 stars 25 forks source link

Consolidate getIncludeData instances? #151

Closed Danp2 closed 1 year ago

Danp2 commented 1 year ago

There are currently two instances of the function getIncludeData. The one in ai_completion.js returns a simple array containing the function names. The one in ai_signature.js returns an object containing additional details.

@loganch Should these be reduced to a single instance located in util.js?

Danp2 commented 1 year ago

I ran some quick tests, and the execution times for the two versions of getIncludeData are nearly identical (the one from ai_signature.js takes about 1ms longer).

Danp2 commented 1 year ago

See #154 if anyone wants to test the changes.