Skip to content
  • brian m. carlson's avatar
    Convert lookup_commit* to struct object_id · bc83266a
    brian m. carlson authored and Junio C Hamano's avatar Junio C Hamano committed
    
    
    Convert lookup_commit, lookup_commit_or_die,
    lookup_commit_reference, and lookup_commit_reference_gently to take
    struct object_id arguments.
    
    Introduce a temporary in parse_object buffer in order to convert this
    function.  This is required since in order to convert parse_object and
    parse_object_buffer, lookup_commit_reference_gently and
    lookup_commit_or_die would need to be converted.  Not introducing a
    temporary would therefore require that lookup_commit_or_die take a
    struct object_id *, but lookup_commit would take unsigned char *,
    leaving a confusing and hard-to-use interface.
    
    parse_object_buffer will lose this temporary in a later patch.
    
    This commit was created with manual changes to commit.c, commit.h, and
    object.c, plus the following semantic patch:
    
    @@
    expression E1, E2;
    @@
    - lookup_commit_reference_gently(E1.hash, E2)
    + lookup_commit_reference_gently(&E1, E2)
    
    @@
    expression E1, E2;
    @@
    - lookup_commit_reference_gently(E1->hash, E2)
    + lookup_commit_reference_gently(E1, E2)
    
    @@
    expression E1;
    @@
    - lookup_commit_reference(E1.hash)
    + lookup_commit_reference(&E1)
    
    @@
    expression E1;
    @@
    - lookup_commit_reference(E1->hash)
    + lookup_commit_reference(E1)
    
    @@
    expression E1;
    @@
    - lookup_commit(E1.hash)
    + lookup_commit(&E1)
    
    @@
    expression E1;
    @@
    - lookup_commit(E1->hash)
    + lookup_commit(E1)
    
    @@
    expression E1, E2;
    @@
    - lookup_commit_or_die(E1.hash, E2)
    + lookup_commit_or_die(&E1, E2)
    
    @@
    expression E1, E2;
    @@
    - lookup_commit_or_die(E1->hash, E2)
    + lookup_commit_or_die(E1, E2)
    
    Signed-off-by: default avatarbrian m. carlson <sandals@crustytoothpaste.net>
    Signed-off-by: default avatarJunio C Hamano <gitster@pobox.com>
    bc83266a