[PATCH 5 of 5] histedit: break _histedit function into smaller pieces (add _continueaction)

Pierre-Yves David pierre-yves.david at ens-lyon.org
Thu Feb 11 22:25:38 EST 2016



On 02/05/2016 11:04 PM, Kostia Balytskyi wrote:
> Sorry, the explanation was meant to be in Patch 0 before I learned that hg mailing list does not like those. The ultimate goal for me is to fix issue4800. I tried to send the easy patch once, but Pierre-Eves did not like it. His suggestion on fixing that requires much deeper understanding of how histedit works from me and while I'm trying to figure that out, I'm refactoring in process. I've been told that it's a good thing :)
>
> On 2/5/16, 9:50 PM, "Augie Fackler" <raf at durin42.com> wrote:
>
>> On Tue, Feb 02, 2016 at 07:28:06PM +0000, Kostia Balytskyi wrote:
>>> # HG changeset patch
>>> # User Kostia Balytskyi <ikostia at fb.com>
>>> # Date 1454348814 0
>>> #      Mon Feb 01 17:46:54 2016 +0000
>>> # Branch stable
>>> # Node ID a6d5d9f5a18ccc250176665cb67f3d018249958a
>>> # Parent  55512493bd388e75f3d6187a9562330ddc3474e9
>>> histedit: break _histedit function into smaller pieces (add _continueaction)
>>
>> For the most part I like this, but where is this going? I'm just not seeing the forest in these trees yet...

I can confirm that Kostia in on good track to make histedit smarter 
about history edition happening during interruption.

As augie said "For the most part I like this" I'll give Augie the 
opportunity to either push it or to request the small change implies by 
the "the most part".

Cheers,

-- 
Pierre-Yves David


More information about the Mercurial-devel mailing list