Re: Patch 52001
sebb <[email protected]> Thu, 13 Oct 2011 18:45:43 +0100
| Newsgroups | gmane.comp.jakarta.cactus.devel |
|---|---|
| Message-ID | <CAOGo0VbXvMB_yFCKPP4Hx=nQmyige2bSWhmD_wv8dmsu0xrA=g@mail.gmail.com> |
On 13 October 2011 18:28, Philippe Mouawad <[email protected]> wrote: > Hello Sebb, > Do you think we should implement issue > 52001<https://issues.apache.org/bugzilla/show_bug.cgi?id=52001> > : > > - Add scroll automatically to Summary Report > > The only use case I see is when you label sampler with a variable name, > otherwise you may scroll but it won't last after some time number of lines > will stay the same. Yes, it's only needed in limited circumstances, which is why I don't really see the need for it. > If you give me the go, I will patch it. It's not my personal decision; we make decisions by consensus. Any committer can veto a change on a technical basis (e.g. it would break something else) but that is probably not the case here. If it were me, I would not apply the patch as I cannot see the point; more code to test and maintain with little benefit. So I don't support the patch, but if you feel it would be useful I won't veto it.