sjbarag / brs

An interpreter for the BrightScript language that runs on non-Roku platforms.
MIT License
114 stars 43 forks source link

fix(interpreter): Support case insensitive names for functions in components #647

Closed alimnios72 closed 3 years ago

alimnios72 commented 3 years ago

In RBI functions with the same name are always overriden (despite the casing) if extending components with the only exception of the Init method which is a special case. With this fix, it doesn't matter what convention you use to name your function as it's going to be converted to lowerCase to fix the override issue we were having. In addition to the fix, I added a test case to functions to verify that casing doesn't matter when naming functions.

Fixes #502