Erlang Mailing Lists

Author Message

<  Erlang patches mailing list  ~  [erlang-bugs] Bug: anonymous gen_event process + format_stat

Guest
Posted: Wed Jul 07, 2010 2:47 pm Reply with quote
Guest
On Sun, Jun 13, 2010 at 02:02:12PM -0700, Geoff Cant wrote:
>
> Hi all, it seems during the last round of format_status improvements a
> bug was introduced for anonymous gen_event processes.
>
> Repro:
> {ok, Pid} = gen_event:start(), sys:get_status(Pid).
>
> Expected: Status report
> Actual: Crash due to calling lists:concat on a list containing a pid()
>
> I've done some work on format_status here:
> git fetch git://github.com/archaelus/otp.git format_status_improvements

Thank you! It will be included in 'pu' as gc/gen-format-status-improvements.

I have squashed the commits and reformatted/reworded the commit comment
according to our standards.

Sorry I missed this one when it was posted. It was on the wrong
list (erlang-bugs) so my radar was on low effect... or something.
Sorry about that. Instead I did some more work on the commit formatting.

>
> I've fixed the bug by porting the gen_fsm StatusHdr code into gen_event
> at 00b032608c3ad3573d2b. Then I refactored the StatusHdr code out of
> gen_server, gen_fsm and gen_event into 'gen' at 9103fa5136d360839aad,
> and extended the test suite to test for the anon gen_event format status
> bug at 480f1282d9bcdd7d40c3.
>
> Cheers,
> --
> Geoff Cant
>

--

/ Raimo Niskanen, Erlang/OTP, Ericsson AB

________________________________________________________________
erlang-patches (at) erlang.org mailing list.
See http://www.erlang.org/faq.html
To unsubscribe; mailto:erlang-patches-unsubscribe@erlang.org

Post received from mailinglist
Guest
Posted: Wed Jul 07, 2010 2:47 pm Reply with quote
Guest
On Sun, Jun 13, 2010 at 02:02:12PM -0700, Geoff Cant wrote:
>
> Hi all, it seems during the last round of format_status improvements a
> bug was introduced for anonymous gen_event processes.
>
> Repro:
> {ok, Pid} = gen_event:start(), sys:get_status(Pid).
>
> Expected: Status report
> Actual: Crash due to calling lists:concat on a list containing a pid()
>
> I've done some work on format_status here:
> git fetch git://github.com/archaelus/otp.git format_status_improvements

Thank you! It will be included in 'pu' as gc/gen-format-status-improvements.

I have squashed the commits and reformatted/reworded the commit comment
according to our standards.

Sorry I missed this one when it was posted. It was on the wrong
list (erlang-bugs) so my radar was on low effect... or something.
Sorry about that. Instead I did some more work on the commit formatting.

>
> I've fixed the bug by porting the gen_fsm StatusHdr code into gen_event
> at 00b032608c3ad3573d2b. Then I refactored the StatusHdr code out of
> gen_server, gen_fsm and gen_event into 'gen' at 9103fa5136d360839aad,
> and extended the test suite to test for the anon gen_event format status
> bug at 480f1282d9bcdd7d40c3.
>
> Cheers,
> --
> Geoff Cant
>

--

/ Raimo Niskanen, Erlang/OTP, Ericsson AB

________________________________________________________________
erlang-patches (at) erlang.org mailing list.
See http://www.erlang.org/faq.html
To unsubscribe; mailto:erlang-patches-unsubscribe@erlang.org

Post received from mailinglist

Display posts from previous:  

All times are GMT
Page 1 of 1
This forum is locked: you cannot post, reply to, or edit topics.

Jump to:  

You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot vote in polls in this forum
You cannot attach files in this forum
You cannot download files in this forum