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

Re: [PATCH 1/2] packfile: drop a repeated enum declaration

From
Jonathan Nieder <jrnieder@gmail.com>
Date
Jul 24, 2018, 21:57 UTC
Message-ID
<20180724215736.GC136514@aiede.svl.corp.google.com>
In-Reply-To
<20180724215223.27516-2-dev+git@drbeat.li>
Hi,
Beat Bolli wrote:
Show 10 quoted lines
> --- a/packfile.h
> +++ b/packfile.h
> @@ -6,7 +6,6 @@
>  /* in object-store.h */
>  struct packed_git;
>  struct object_info;
> -enum object_type;
>  
>  /*
>   * Generate the filename to be used for a pack file with checksum "sha1" and

Good catch. This means packfile.h should #include "cache.h" --- otherwise, it is not usable in a file that does

	#include "git-compat-util.h"
	#include "packfile.h"

Thanks and hope that helps, Jonathan

Previous: Beat Bolli
Message 8 of 8 in “Pedantic fixes for Apple clang”
  1. 0/2 Pedantic fixes for Apple clangBeat Bolli, Jul 24, 2018
  2. 2/2 remote-odb: un-inline function remote_odb_reinitBeat Bolli, Jul 24, 2018
  3. Jonathan NiederJul 24, 2018
  4. Beat BolliJul 24, 2018
  5. Christian CouderJul 25, 2018
  6. Junio C HamanoJul 25, 2018
  7. 1/2 packfile: drop a repeated enum declarationBeat Bolli, Jul 24, 2018
  8. Jonathan NiederJul 24, 2018

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.