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

AtU8 UTF8 atoms chunk decompilation #777

Merged
merged 5 commits into from
Aug 7, 2017
Merged

AtU8 UTF8 atoms chunk decompilation #777

merged 5 commits into from
Aug 7, 2017

Conversation

KronicDeth
Copy link
Owner

@KronicDeth KronicDeth commented Aug 7, 2017

Resolves #772

Changelog

Enhancements

  • Include specific message for each decompilation error reason.
  • Regression test for Decompilation not working with OTP 20 using AtU8 chunk for ASCII atoms #772
  • AtU8 UTF8 atoms chunk decompilation to support OTP 20 compiled .beam files.
  • Regression test for Elixir 1.5.0 announcement to ensure that unicode from AtU8 chunk can be read from beam file produced by following Elixir code:
defmodule AtU8Test do
  def こんにちは世界 do
    :こんにちは世界
  
     saudação = "Bom dia!"

    saudação
  end
end

Bug Fixes

  • Decompilated -> Decompilation

@KronicDeth KronicDeth added this to the v6.0.0 milestone Aug 7, 2017
@KronicDeth KronicDeth self-assigned this Aug 7, 2017
Check for old Atom, if it doesn't exist, check for AtU8 chunk.
Repository owner deleted a comment Aug 7, 2017
Repository owner deleted a comment Aug 7, 2017
Ensure that unicode from AtU8 chunk can be read from beam file produced
by following Elixir code:

```elixir
defmodule AtU8Test do
  def こんにちは世界 do
    :こんにちは世界

    saudação = "Bom dia!"

    saudação
  end
end
```
@KronicDeth KronicDeth merged commit 1f34fa3 into master Aug 7, 2017
@KronicDeth KronicDeth deleted the 772 branch August 7, 2017 04:50
@KronicDeth KronicDeth mentioned this pull request Aug 7, 2017
2 tasks
KronicDeth added a commit that referenced this pull request Aug 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant