Re: [PING][PATCH] Add shortcuts for "nexti" and "stepi" commands in Single-Key mode

[Simon Marchi <simon.marchi@polymtl.ca>]

On 2017-07-27 07:59, Ruslan Kabatsayev wrote:
> On 26 July 2017 at 23:53, Simon Marchi <simon.marchi@polymtl.ca> wrote:
>> On 2017-07-26 21:19, Ruslan Kabatsayev wrote:
>>> 
>>> So, the patch seems to have been approved by one reviewer (see [1]). 
>>> But
>>> it
>>> doesn't appear to have been pushed, nor do I have any further 
>>> instructions
>>> on how to get it merged. What should I do?
>> 
>> 
>> Hi Ruslan,
>> 
>> The patch is fine with me too, it seems to work well and it's a good 
>> idea.
>> 
>> It might be a bit more work to implement (an idea for another patch 
>> maybe),
>> but do you think it would be nice if n and s did next and step when 
>> focusing
>> a source window, but did nexti and stepi when focusing an asm window?  
>> When
>> in an asm window, I think you will much more likely want to use 
>> nexti/stepi
>> than next/step, so maybe they should be bound to the n and s rather 
>> than N
>> and S in that case?
> This sounds like a great idea. If this is implemented, do you think N
> and S are then useless and confusing (since n/s would be
> focus-dependent while N/S won't)?

I was thinking that perhaps that when focusing...

a source view: s/n do source line stepping, S/N do assembly stepping
an asm view: s/n do assembly stepping, S/N do source line stepping

Basically, they would switch role.  Is that too confusing?

>> 
>> In order to get the patch in, you have two choices: if you plan to
>> contribute more in the future, we can get you write access to the 
>> repo, so
>> that you can push your patches in once they are approved.  Otherwise, 
>> we can
>> push it for you.  Let me know what you prefer.
> Yes, I do plan to contribute in the future.

I'll send you the details in private.

Thanks,

Simon