Skip to content
Commits on Oct 5, 2011
  1. @mhagger @gitster

    Change check_ref_format() to take a flags argument

    mhagger committed with gitster Sep 15, 2011
    Change check_ref_format() to take a flags argument that indicates what
    is acceptable in the reference name (analogous to "git
    check-ref-format"'s "--allow-onelevel" and "--refspec-pattern").  This
    is more convenient for callers and also fixes a failure in the test
    suite (and likely elsewhere in the code) by enabling "onelevel" and
    "refspec-pattern" to be allowed independently of each other.
    
    Also rename check_ref_format() to check_refname_format() to make it
    obvious that it deals with refnames rather than references themselves.
    
    Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
    Signed-off-by: Junio C Hamano <gitster@pobox.com>
Commits on Nov 29, 2010
  1. @gitster

    commit: Add commit_list prefix in two function names.

    Thiago Farina committed with gitster Nov 26, 2010
    Add commit_list prefix to insert_by_date function and to sort_by_date,
    so it's clear that these functions refer to commit_list structure.
    
    Signed-off-by: Thiago Farina <tfransosi@gmail.com>
    Signed-off-by: Junio C Hamano <gitster@pobox.com>
Commits on Jul 23, 2009
  1. @gitster

    janitor: useless checks before free

    Pierre Habouzit committed with gitster Jul 22, 2009
    Signed-off-by: Pierre Habouzit <madcoder@debian.org>
    Signed-off-by: Junio C Hamano <gitster@pobox.com>
Commits on Feb 11, 2009
  1. @gitster

    Drop double-semicolon in C

    gitster committed Feb 10, 2009
    The worst offenders are "continue;;" and "break;;" in switch statements.
    
    Signed-off-by: Junio C Hamano <gitster@pobox.com>
Commits on Oct 18, 2008
  1. @gitster

    make alloc_ref_from_str() the new alloc_ref()

    René Scharfe committed with gitster Oct 18, 2008
    With all calls to alloc_ref() gone, we can remove it and then we're free
    to give alloc_ref_from_str() the shorter name.  It's a much nicer
    interface, as the callers always need to have a name string when they
    allocate a ref anyway and don't need to calculate and pass its length+1
    any more.
    
    Signed-off-by: Rene Scharfe <rene.scharfe@lsrfire.ath.cx>
    Signed-off-by: Junio C Hamano <gitster@pobox.com>
Commits on Oct 12, 2008
  1. @spearce

    Replace calls to strbuf_init(&foo, 0) with STRBUF_INIT initializer

    Brandon Casey committed with spearce Oct 9, 2008
    Many call sites use strbuf_init(&foo, 0) to initialize local
    strbuf variable "foo" which has not been accessed since its
    declaration. These can be replaced with a static initialization
    using the STRBUF_INIT macro which is just as readable, saves a
    function call, and takes up fewer lines.
    
    Signed-off-by: Brandon Casey <casey@nrlssc.navy.mil>
    Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Commits on Jun 4, 2008
  1. @peff @gitster

    Fix "git clone http://$URL" to check out the worktree when asked

    peff committed with gitster Jun 4, 2008
    The builtin-clone now does the http commit walking and the tree unpacking
    in the same process, and the commit walker leaves the in-core objects in a
    funny state.  When forgetting the data read from the tree object, the
    object should be marked "not parsed yet" for later users.
    
    Acked-by: Linus Torvalds <torvalds@linux-foundation.org>
    Signed-off-by: Junio C Hamano <gitster@pobox.com>
Commits on May 11, 2008
  1. @kjk @gitster

    alloc_ref_from_str(): factor out a common pattern of alloc_ref from s…

    kjk committed with gitster May 10, 2008
    …tring
    
    Also fix an underallocation in walker.c::interpret_target().
    
    Signed-off-by: Krzysztof Kowalczyk <kkowalczyk@gmail.com>
    Signed-off-by: Junio C Hamano <gitster@pobox.com>
Commits on Apr 27, 2008
  1. @gitster

    Make walker.fetch_ref() take a struct ref.

    Daniel Barkalow committed with gitster Apr 26, 2008
    This simplifies a few things, makes a few things slightly more
    complicated, but, more importantly, allows that, when struct ref can
    represent a symref, http_fetch_ref() can return one.
    
    Incidentally makes the string that http_fetch_ref() gets include "refs/"
    (if appropriate), because that's how the name field of struct ref works.
    As far as I can tell, the usage in walker:interpret_target() wouldn't have
    worked previously, if it ever would have been used, which it wouldn't
    (since the fetch process uses the hash instead of the name of the ref
    there).
    
    Signed-off-by: Daniel Barkalow <barkalow@iabervon.org>
    Signed-off-by: Junio C Hamano <gitster@pobox.com>
Commits on Feb 26, 2008
  1. @gitster

    Remove unused object-ref code

    Martin Koegler committed with gitster Feb 25, 2008
    Signed-off-by: Martin Koegler <mkoegler@auto.tuwien.ac.at>
    Signed-off-by: Junio C Hamano <gitster@pobox.com>
Commits on Dec 18, 2007
  1. @samv @gitster

    Clarify error response from 'git fetch' for bad responses

    samv committed with gitster Dec 18, 2007
    This error message prints the reponse from the server at this point.
    Label it as such in the output.
    
    Signed-off-by: Sam Vilain <sam@vilain.net>
    Signed-off-by: Junio C Hamano <gitster@pobox.com>
Commits on Oct 16, 2007
  1. @spearce

    Merge branch 'master' into db/fetch-pack

    spearce committed Oct 16, 2007
    There's a number of tricky conflicts between master and
    this topic right now due to the rewrite of builtin-push.
    Junio must have handled these via rerere; I'd rather not
    deal with them again so I'm pre-merging master into the
    topic.  Besides this topic somehow started to depend on
    the strbuf series that was in next, but is now in master.
    It no longer compiles on its own without the strbuf API.
    
    * master: (184 commits)
      Whip post 1.5.3.4 maintenance series into shape.
      Minor usage update in setgitperms.perl
      manual: use 'URL' instead of 'url'.
      manual: add some markup.
      manual: Fix example finding commits referencing given content.
      Fix wording in push definition.
      Fix some typos, punctuation, missing words, minor markup.
      manual: Fix or remove em dashes.
      Add a --dry-run option to git-push.
      Add a --dry-run option to git-send-pack.
      Fix in-place editing functions in convert.c
      instaweb: support for Ruby's WEBrick server
      instaweb: allow for use of auto-generated scripts
      Add 'git-p4 commit' as an alias for 'git-p4 submit'
      hg-to-git speedup through selectable repack intervals
      git-svn: respect Subversion's [auth] section configuration values
      gtksourceview2 support for gitview
      fix contrib/hooks/post-receive-email hooks.recipients error message
      Support cvs via git-shell
      rebase -i: use diff plumbing instead of porcelain
      ...
    
    Conflicts:
    
    	Makefile
    	builtin-push.c
    	rsh.c
Commits on Sep 19, 2007
  1. @gitster

    Modularize commit-walker

    Daniel Barkalow committed with gitster Sep 10, 2007
    This turns the extern functions to be provided by the backend into a
    struct of pointers, renames the functions to be more
    namespace-friendly, and updates http-fetch to this interface. It
    removes the unused include from http-push.c. It makes git-http-fetch a
    builtin (with the implementation a separate file, accessible
    directly).
    
    Signed-off-by: Daniel Barkalow <barkalow@iabervon.org>
    Signed-off-by: Junio C Hamano <gitster@pobox.com>
Something went wrong with that request. Please try again.