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

Re: [PATCH] Use DIFF_XDL_SET/DIFF_OPT_SET instead of raw bit-masking

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Feb 17, 2009, 12:05 UTC
Message-ID
<alpine.DEB.1.00.0902171304130.6185@intel-tinevez-2-302>
In-Reply-To
<1234841209-3960-1-git-send-email-keith@cs.ucla.edu>
Hi,
On Mon, 16 Feb 2009, Keith Cascio wrote:
> 
> Signed-off-by: Keith Cascio <keith@cs.ucla.edu>
> ---
Rationale?

If you are going to add something on top of that, I can understand that, but this patch is not labeled [1/n]. And...

>  diff.h |    3 +++
>  diff.c |   17 ++++++++++-------
>  2 files changed, 13 insertions(+), 7 deletions(-)

... this does not look good to me, without a compelling reason why we want to have the patch nevertheless.

Ciao, Dscho

Previous: Keith CascioNext: Keith Cascio
Message 2 of 8 in “Use DIFF_XDL_SET/DIFF_OPT_SET instead of raw bit-masking”
  1. Use DIFF_XDL_SET/DIFF_OPT_SET instead of raw bit-maskingKeith Cascio, Feb 17, 2009
  2. Johannes SchindelinFeb 17, 2009
  3. Keith CascioFeb 17, 2009
  4. Johannes SchindelinFeb 17, 2009
  5. Keith CascioFeb 18, 2009
  6. Johannes SchindelinFeb 18, 2009
  7. Keith CascioFeb 18, 2009
  8. Jeff KingFeb 19, 2009

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.