• Show log

    Commit

  • Hash : 2e0a3048
    Author : Patrick Steinhardt
    Date : 2019-01-23T10:48:55

    oidmap: introduce high-level setter for key/value pairs
    
    Currently, one would use either `git_oidmap_insert` to insert key/value pairs
    into a map or `git_oidmap_put` to insert a key only. These function have
    historically been macros, which is why their syntax is kind of weird: instead of
    returning an error code directly, they instead have to be passed a pointer to
    where the return value shall be stored. This does not match libgit2's common
    idiom of directly returning error codes.Furthermore, `git_oidmap_put` is tightly
    coupled with implementation details of the map as it exposes the index of
    inserted entries.
    
    Introduce a new function `git_oidmap_set`, which takes as parameters the map,
    key and value and directly returns an error code. Convert all trivial callers of
    `git_oidmap_insert` and `git_oidmap_put` to make use of it.