Re: [PATCH 3/9] t1300: avoid relying on a bug

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Mar 30, 2018, 12:38 UTC
Message-ID: <nycvar.QRO.7.76.6.1803301438120.5026@qfpub.tvgsbejvaqbjf.bet>
In-Reply-To: <20180329194313.GD2939@sigill.intra.peff.net>

Hi Peff,

On Thu, 29 Mar 2018, Jeff King wrote:

Show 11 quoted lines

> On Thu, Mar 29, 2018 at 05:18:45PM +0200, Johannes Schindelin wrote:
> 
> > The test case 'unset with cont. lines' relied on a bug that is about to
> > be fixed: it tests *explicitly* that removing the last entry from a
> > config section leaves an *empty* section behind.
> > 
> > Let's fix this test case not to rely on that behavior, simply by
> > preventing the section from becoming empty.
> 
> Seems like a good solution. I don't think we care in particular about
> testing a multi-line value at the end of the file.

... and if we did, we should have documented that.

Ciao, Dscho

$ 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.