git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] gitk: fix the context menu not appearing in the presence of submodule diffs

From
Роман Донченко <dpb@corrigendum.ru>
Date
Apr 29, 2020, 23:11 UTC
Message-ID
<58d9c60c-1b0c-ef59-0355-da6edcbc8cdf@corrigendum.ru>
In-Reply-To
<20200413051151.GB14888@blackberry>
Hi,

I just checked the repository, and it doesn't seem like it's actually applied...

-Roman.
13.04.2020 8:11, Paul Mackerras пишет:
Show 24 quoted lines
> On Sun, Dec 15, 2019 at 09:25:50PM +0300, Роман Донченко wrote:
>> Currently, submodule diffs can cause the diff context menu to fail
>> to appear because of a couple bugs in parseblobdiffline:
>>
>> * it appends the submodule name to ctext_file_lines instead of
>>    a line number, which breaks the binary search in find_ctext_fileinfo;
>>
>> * it can desynchronize ctext_file_names and ctext_file_lines
>>    by appending to the former but not the latter, which also breaks
>>    find_ctext_fileinfo.
>>
>> Fix both of these.
>>
>> Note: a side effect of this patch is that the context menu also
>> starts appearing when you right-click on submodule diffs (and not just
>> regular diffs). The menu is non-functional in this case, though,
>> since you can't run blame on submodules.
>>
>> Signed-off-by: Роман Донченко <dpb@corrigendum.ru>
> 
> Thanks, applied.
> 
> Paul.
> 
Previous: Paul Mackerras
Message 3 of 3 in “gitk: fix the context menu not appearing in the presence of submodule diffs”
  1. gitk: fix the context menu not appearing in the presence of submodule diffsРоман Донченко, Dec 15, 2019
  2. Paul MackerrasApr 13, 2020
  3. Роман ДонченкоApr 29, 2020

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.