Hello Andrey, I made some changes to your patch that I reuploaded on BugZilla issue, feel free to check it, modify it .... Changes are the following:
- Added a limit on the history (25 it should be configurable), to avoid OutOfMemoryError when Test Plan is big - Added Pause/Resume methods that are used during loading , closing and merging of Tree, cause if we don't add it (at least for close and loading): 1) It does not seem useful to be able to undo elements that were added during load phase 2) Performance are really bad , a big test plan (6 Mo) takes more than 30 seconds to load - There is one drawback to previous change, it's that we won't be able to undo merged part (right click>merge) - Added javadocs - patch to r1231672 Users are welcome to indicate if this is a big drawback. Regards Philippe http://www.ubik-ingenierie.com On Fri, Jan 27, 2012 at 12:06 PM, Philippe Mouawad < [email protected]> wrote: > Thank you Andrey, > > We will try to integrate it as soon as possible. > > Regards > Philippe > > > On Fri, Jan 27, 2012 at 11:29 AM, Shay Ginsbourg <[email protected]>wrote: > >> Very well done! >> >> ------Original Message------ >> From: Oliver Lloyd >> To: [email protected] >> ReplyTo: JMeter Users List >> Subject: Re: CTRL+Z as undo >> Sent: Jan 27, 2012 12:13 >> >> Andrey, thanks for this, you're a superstar. >> >> ----- >> http://www.http503.com/ >> -- >> View this message in context: >> http://jmeter.512774.n5.nabble.com/CTRL-Z-as-undo-tp4848249p5434931.html >> Sent from the JMeter - User mailing list archive at Nabble.com. >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: [email protected] >> For additional commands, e-mail: [email protected] >> >> >> >> Sent from my BlackBerry® smartphone from orange >> > > > > -- > Cordialement. > Philippe Mouawad. > > > > -- Cordialement. Philippe Mouawad.
