Fix case_clause error #1666

lukebakken · 2017-12-20T20:34:20Z

This fixes the following error when running observer:start()

(rabbit@shostakovich)23> observer:start().
{error,{{case_clause,[]},
        [{observer_pro_wx,init_table_holder,3,
                          [{file,"observer_pro_wx.erl"},{line,466}]}]}}

This fixes the following errors when running `observer:start()` ``` (rabbit@shostakovich)23> observer:start(). {error,{{case_clause,[]}, [{observer_pro_wx,init_table_holder,3, [{file,"observer_pro_wx.erl"},{line,466}]}]}} ```

CLAassistant · 2017-12-20T20:34:26Z

All committers have signed the CLA.

dgud · 2018-01-04T14:46:11Z

How have you managed to get an empty list there?
The config is get via get_config call which does a get_accum call that always returns a boolean.
I'm confused, what is it that I'm missing?

lukebakken · 2018-01-04T15:08:19Z

🤷‍♂️ I'm not sure, I only reported what I saw and what I changed to fix it. I should have been more complete when reporting my environment -

Arch Linux
Erlang 19.3, 20.2 both built by kerl

What is interesting is that I can no longer reproduce this issue so there must have been an update to wkgtk or some other component that fixed it. I'll close this but if it happens again I'll provide more information as well as some traces.

dgud · 2018-01-04T15:54:33Z

This have nothing to do with graphics it is just erlang code, so it should not depend on wx.

I can only see it happen if you manually edit the config file... or if you "beta" tested and
saved some old config format before the 20.0 release (where saving config automagially was
implemented) but there is only one commit (now) with all the changes regarding config.
So from I what I can see now, it should not happen :-)

mobileoverlord · 2018-03-07T19:48:00Z

I am currently experiencing this on my machine. I am installing erlang through kerl by means of asdf I have uninstalled and reinstalled wx tools and erlang and I still cannot launch observer. I am currently running 20.2.4. There are times where asdf will switch between OTP 19 / 20 depending on what the requirements of the project I am working on. You mention a config file that could be an issue. Is this file written globally?

mobileoverlord · 2018-03-07T19:59:14Z

okay, I found the file and renamed it. Now it works. Let me know if you would like more information on this.

dgud · 2018-03-08T07:16:29Z

So I must be missing something, I thought I added the "config file" functionality in OTP-20,
but it seems to be a problem when switching between 19 and 20 then.
Can you email me that config file dgud AT erlang.org.

Fix case_clause error

e1d8710

This fixes the following errors when running `observer:start()` ``` (rabbit@shostakovich)23> observer:start(). {error,{{case_clause,[]}, [{observer_pro_wx,init_table_holder,3, [{file,"observer_pro_wx.erl"},{line,466}]}]}} ```

uabboli added the team:MW label Dec 22, 2017

uabboli assigned dgud Dec 22, 2017

lukebakken closed this Jan 4, 2018

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix case_clause error #1666

Fix case_clause error #1666

lukebakken commented Dec 20, 2017

CLAassistant commented Dec 20, 2017 •

edited

Loading

dgud commented Jan 4, 2018

lukebakken commented Jan 4, 2018

dgud commented Jan 4, 2018

mobileoverlord commented Mar 7, 2018 •

edited

Loading

mobileoverlord commented Mar 7, 2018

dgud commented Mar 8, 2018

Fix case_clause error #1666

Fix case_clause error #1666

Conversation

lukebakken commented Dec 20, 2017

CLAassistant commented Dec 20, 2017 • edited Loading

dgud commented Jan 4, 2018

lukebakken commented Jan 4, 2018

dgud commented Jan 4, 2018

mobileoverlord commented Mar 7, 2018 • edited Loading

mobileoverlord commented Mar 7, 2018

dgud commented Mar 8, 2018

CLAassistant commented Dec 20, 2017 •

edited

Loading

mobileoverlord commented Mar 7, 2018 •

edited

Loading