andytill / erlyberly

erlang tracing for the masses
https://twitter.com/erlyberlytips
GNU General Public License v3.0
693 stars 43 forks source link

Fix Elixir module names #140

Closed aboroska closed 7 years ago

aboroska commented 7 years ago

Display MyMod.MySubMod instead of :"Elixir.MyMod.Mysubmod"

andytill commented 7 years ago

Great feature.

This change also changes how atoms are shown in the trace view. I think it should only change the module name in the list of modules, since elixir does not have special handling of atoms starting with Elixir. other than for function calls.

aboroska commented 7 years ago

Not only function calls related. BigWords in Elixir syntax will a prefixed atom at the beam level: 'Elixir.BigWords' in Erlang syntax. Everywhere.

Try tracing in Erlang syntax:

iex(my@localhost)1> Application.put_env :a, :b, "hello"
:ok
iex(my@localhost)2> Application.put_env :a, :b, BigWords
:ok
iex(my@localhost)3> Application.put_env :a, :b, BigWords.Hello
:ok

screen shot 2017-01-29 at 12 26 20

andytill commented 7 years ago

Thanks, I didn't know that.