Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed roDeviceInfo method type and some test cases #689

Closed

Conversation

lvcabral
Copy link
Contributor

The method getAudioDecodeInfo was wrongly spelled, and getSupportedGraphicsResolutions was returning an AA when the correct is a roArray.

Skipped one roXMLElement test case that is not behaving as Roku, the implementation is wrong, to be fixed later when roXMLList is implemented.

TwitchBronBron and others added 30 commits April 19, 2023 10:49
refactor in preparation for adoption the project
…ot-compliant

Fixed `val()` edge cases: hex without radix and `NaN`
* add logic for optional chaining

* remove console log

* format doc

* fix test

* fix uts
* Preventing multiple calls for dot-chained methods

* Fixed `prettier` issue
…d `roTimespan` #28 (#29)

* fix(components): Replacing package luxon by day.js on `roDateTime` and `roTimespan` #28

* Fixed prettier issue
…nd `roAssociativeArray` (#33)

* Implements missing methods from `ifEnum` in `roArray` and `roAssociativeArray` #26

* Implemented unit tests

* Prettier fix

---------

Co-authored-by: Bronley Plumb <[email protected]>
* feat(lex): Emit try/catch/throw lexemes for parsing

RBI 9.4 adds support for error handling via the `try`/`catch` model [1].
Interestingly, `try`, `catch`, and `endtry` are all valid identifiers!
That makes parsing a little bit tougher, but it's not unprecedented in
this language.  Detect `try`/`catch`/throw`/`end try` during lexing and
emit the proper lexemes.

[1] https://developer.roku.com/docs/references/brightscript/language/error-handling.html
see sjbarag#554

* feat(parse): Consume try, catch, and end try lexemes

Throwing exceptions and catching them aren't yet supported in `brs`, so
executing only the `try` block seems to be a reasonable "bare minimum"
approach to try/catch support.  Handle `try`, `catch`, and `end try`
lexemes in the parser, emitting TryCatch statements for the interpreter
to execute naïvely.

see sjbarag#554

* Fixed Test Case

---------

Co-authored-by: Sean Barag <[email protected]>
Co-authored-by: Bronley Plumb <[email protected]>
* Adds create-package CI build for quicker iteration

* fix illegal URL chars in name

* use - instead of _

* better release title
* fix(lib): Component XML path parsing was failing on edge case

* Fixed search string to match what `fast-glob` supports
lvcabral and others added 19 commits December 11, 2023 08:50
…()` are not properly boxing parameters (#42)
…th()` (#44)

* Fixed # 43 - Implemented `roString` methods `startsWith()` and `endsWith()`

* Fixed test code formatting
* Fixed path handling for Windows

* Fixed code formatting

* Improved Posix normalization
* Implemented component `roList`

* Fixed test code formatting

* Fixed `ifString.tokenize()` to match Roku behavior

* Fixed code formatting

* Replaces for loop by a for-of loop
* Added support for bitwise NOT

* Prettier Fixes
* Added option to disable colors and fixed unit tests to not fail with colors

* Implemented `roPath` and fixed comparison to match Roku behavior

* Added unit tests and fixed prettier issues

* Properly compare Number with Boolean

* Added support to AND and OR between Boolean and Numbers

* Updated unit tests and fixed Int64

* Improved URL regex for CLI colorizer
* Implemented `roByteArray` component

* Added unit tests and fixed issues on roByteArray

* Fixes to `roArray` and `roList`

* Removed empty line from a test file
* Implemented `Box()` function and improved boxing on Numbers and Boolean

* Fixed lint issue

* Fixed Invalid check when on the left side does not work (boxing issue) #7

* Moved `debugLocalVariables()` to interpreter (to be used by MicroDebugger in the future)

* Fixed `-n, --no-color` CLI option

* Documented `debugLocalVariables()`
* Implemented `ifArraySizeInfo` in `roArray`

* Removing empty line

* Fixed warning message
@lvcabral lvcabral closed this Apr 27, 2024
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.

3 participants