Skip to content

Conversation

@akosthekiss
Copy link
Member

  • ERR_SYSCALL is a legacy that has not been used anymore since
    the removal of jerry-libc. This commit removes it from the port
    API.
  • ERR_DISABLED_BYTE_CODE is a (relatively) recent addition to the
    fatal code enum. At the time it was added, the documentation has
    not been updated. This commit adds the missing documentation.
    (Plus, it removes a superfluous JERRY_UNREACHABLE from after a
    jerry_fatal (ERR_DISABLED_BYTE_CODE).)

Note: As the port API is modified, this is an API-breaking change.

JerryScript-DCO-1.0-Signed-off-by: Akos Kiss akiss@inf.u-szeged.hu

- `ERR_SYSCALL` is a legacy that has not been used anymore since
  the removal of jerry-libc. This commit removes it from the port
  API.
- `ERR_DISABLED_BYTE_CODE` is a (relatively) recent addition to the
  fatal code enum. At the time it was added, the documentation has
  not been updated. This commit adds the missing documentation.
  (Plus, it removes a superfluous `JERRY_UNREACHABLE` from after a
  `jerry_fatal (ERR_DISABLED_BYTE_CODE)`.)

Note: As the port API is modified, this is an API-breaking change.

JerryScript-DCO-1.0-Signed-off-by: Akos Kiss akiss@inf.u-szeged.hu
@akosthekiss akosthekiss added enhancement An improvement api Related to the public API documentation Related to documentation jerry-port Related to the port API or the default port implementation labels Jan 3, 2019
@LaszloLango LaszloLango added this to the Release 2.0 milestone Jan 3, 2019
Copy link
Contributor

@LaszloLango LaszloLango left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@robertsipka robertsipka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@robertsipka robertsipka merged commit 1f204de into jerryscript-project:master Jan 3, 2019
@akosthekiss akosthekiss deleted the maintain-fatal-code branch January 3, 2019 14:43
@haochenli
Copy link

Hi there.. @akosthekiss. I am using jerryscript's snapshot with google's benchmark. I counter this error ERR_DISABLED_BYTE_CODE but have no idea what is going on.. also nothing with google, could you please give me a hint of what does this error mean?

@akosthekiss
Copy link
Member Author

@haochenli We are happy to help with errors encountered -- but not here. This is a PR closed 2 years ago, unrelated to the question. If you found a problem, please, open a new issue and give all the requested details (given in the issue template). Then we can discuss things there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api Related to the public API documentation Related to documentation enhancement An improvement jerry-port Related to the port API or the default port implementation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants