[erlang-patches] format_status analog for error_info
Steve Vinoski
vinoski@REDACTED
Mon Mar 1 19:32:47 CET 2010
On Mon, Mar 1, 2010 at 1:16 PM, Andrew Thompson <andrew@REDACTED> wrote:
> On Sun, Feb 28, 2010 at 01:13:37AM -0500, Steve Vinoski wrote:
> > This patch is ready:
> >
> > git fetch git://github.com/vinoski/otp.git format_status_error_info
> >
> > Since gen_event did not previously have format_status callback support, I
> > made these changes in two commits: the first to add termination support
> to
> > existing format_status support in gen_fsm and gen_server along with doc
> and
> > unit tests, and the second commit to add the format_status callback
> support
> > to gen_event along with related doc and unit tests.
> >
>
> The patch looks good to me, except that I *think* (github makes it hard
> to tell) that the indenting on line 545 of gen_fsm.erl is wrong
> (FmtStateData = .. end should line up with the call to error_info).
>
Thanks for checking. Because the FmtStateData = ... is a continuation over
multiple lines, the second and subsequent lines are indented a little beyond
the initial line, which I think is correct. I'm using a fairly recent emacs
erlang-mode without modification or specialization, so I assume it's
indenting correctly.
> The documentation changes look good too. Thanks for your work on this.
>
Sure, and thanks for proposing the idea in the first place.
--steve
More information about the erlang-patches
mailing list