git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
* [PATCH 0/6] miscellaneous patches
@ 2013-10-06 20:47 Ramsay Jones
  2013-10-14 23:25 ` Jonathan Nieder
  0 siblings, 1 reply; 3+ messages in thread
From: Ramsay Jones @ 2013-10-06 20:47 UTC (permalink / raw)
  To: Jonathan Nieder; +Cc: GIT Mailing-list


Hi Jonathan,

These patches don't have too much in common, hence the subject
line, except perhaps that 4 of them fix sparse warnings.

Note that the fourth patch is actually a simplified version of
an earlier RFC patch. Junio didn't like the idea of using a
build variable (GIT_TEST_HIGHLIGHT_BIN) to set the path to the
'correct' highlight tool. see:
http://article.gmane.org/gmane.comp.version-control.git/234138

Having recently had to re-configure Thunderbird, I noticed that
the documentation needed updating. (actually, I was sure that I
had sent a similar patch, years ago, when I last did this! :-P )

ATB,
Ramsay Jones

Ramsay Jones (6):
  config.c: Fix a sparse warning
  refs.c: Fix a sparse warning
  wrapper.c: Fix a sparse warning
  t9500-*.sh: Fix highlight test hang on Linux Mint
  git-format-patch.txt: Add to Thunderbird configuration
  sparse: Fix some "using sizeof on a function" warnings

 Documentation/git-format-patch.txt     | 3 ++-
 Makefile                               | 3 +++
 config.c                               | 2 +-
 refs.c                                 | 2 +-
 t/t9500-gitweb-standalone-no-errors.sh | 6 ++++--
 wrapper.c                              | 2 ++
 6 files changed, 13 insertions(+), 5 deletions(-)

-- 
1.8.4

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 0/6] miscellaneous patches
  2013-10-06 20:47 [PATCH 0/6] miscellaneous patches Ramsay Jones
@ 2013-10-14 23:25 ` Jonathan Nieder
  2013-10-15 14:41   ` Ramsay Jones
  0 siblings, 1 reply; 3+ messages in thread
From: Jonathan Nieder @ 2013-10-14 23:25 UTC (permalink / raw)
  To: Ramsay Jones; +Cc: GIT Mailing-list

Ramsay Jones wrote:

> These patches don't have too much in common, hence the subject
> line, except perhaps that 4 of them fix sparse warnings.

Thanks.  These look good.

I tweaked the descriptions a bit to focus on what sparse was warning
about instead of our having quieted sparse. :)

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 0/6] miscellaneous patches
  2013-10-14 23:25 ` Jonathan Nieder
@ 2013-10-15 14:41   ` Ramsay Jones
  0 siblings, 0 replies; 3+ messages in thread
From: Ramsay Jones @ 2013-10-15 14:41 UTC (permalink / raw)
  To: Jonathan Nieder; +Cc: GIT Mailing-list

On 15/10/13 00:25, Jonathan Nieder wrote:
> Ramsay Jones wrote:
> 
>> These patches don't have too much in common, hence the subject
>> line, except perhaps that 4 of them fix sparse warnings.
> 
> Thanks.  These look good.
> 
> I tweaked the descriptions a bit to focus on what sparse was warning
> about instead of our having quieted sparse. :)
> 

Yes, the subject lines are much better. Thanks!

ATB,
Ramsay Jones

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-10-15 14:42 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-10-06 20:47 [PATCH 0/6] miscellaneous patches Ramsay Jones
2013-10-14 23:25 ` Jonathan Nieder
2013-10-15 14:41   ` Ramsay Jones

Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/git.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).