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

Re: [GSoC][PATCH] merge: use reverse_commit_list() for list reversal

From
Kousik Sanagavarapu <five231003@gmail.com>
Date
Feb 3, 2023, 17:30 UTC
Message-ID
<20230203173006.123249-1-five231003@gmail.com>
In-Reply-To
<xmqq1qn7sm49.fsf@gitster.g>

When I submitted the patch, I didn't realize that reverse_commit_list() helper function had such behavior. It was very different from what I had expected. Maybe I should have been more careful and looked into the details before submitting the patch.

Thanks for your explanation, it was valuable. Especially on the part where you explained about the try_merge_strategy() function, it helped me gain more insight on the whole situation. Sorry for this patch, I will be more careful next time.

P.S. Please ignore the empty email, it was me trying to figure out git-send-email and how could to reply to specific lines of the email by referencing them. The email got sent by accident.

Previous: Kousik SanagavarapuNext: Elijah Newren
Message 4 of 10 in “merge: use reverse_commit_list() for list reversal”
  1. Kousik SanagavarapuFeb 2, 2023
  2. Junio C HamanoFeb 2, 2023
  3. Kousik SanagavarapuFeb 3, 2023
  4. Kousik SanagavarapuFeb 3, 2023
  5. Elijah NewrenFeb 2, 2023
  6. Kousik SanagavarapuFeb 3, 2023
  7. Elijah NewrenFeb 3, 2023
  8. Junio C HamanoFeb 3, 2023
  9. Kousik SanagavarapuFeb 5, 2023
  10. Kousik SanagavarapuFeb 5, 2023

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.