git/notes.c, branch v2.16.2

Merge branch 'rs/hex-to-bytes-cleanup'

2017-11-09T05:31:27Z

Code cleanup. * rs/hex-to-bytes-cleanup: sha1_file: use hex_to_bytes() http-push: use hex_to_bytes() notes: move hex_to_bytes() to hex.c and export it

notes: move hex_to_bytes() to hex.c and export it

2017-11-01T01:35:35Z

Make the function for converting pairs of hexadecimal digits to binary available to other call sites. Signed-off-by: Rene Scharfe Signed-off-by: Junio C Hamano

refs: convert read_ref and read_ref_full to object_id

2017-10-16T02:05:50Z

All but two of the call sites already have parameters using the hash parameter of struct object_id, so convert them to take a pointer to the struct directly. Also convert refs_read_refs_full, the underlying implementation. Signed-off-by: brian m. carlson Signed-off-by: Junio C Hamano

Merge branch 'mh/notes-cleanup'

2017-09-19T01:47:52Z

Code clean-up. * mh/notes-cleanup: load_subtree(): check that `prefix_len` is in the expected range load_subtree(): declare some variables to be `size_t` hex_to_bytes(): simpler replacement for `get_oid_hex_segment()` get_oid_hex_segment(): don't pad the rest of `oid` load_subtree(): combine some common code get_oid_hex_segment(): return 0 on success load_subtree(): only consider blobs to be potential notes load_subtree(): check earlier whether an internal node is a tree entry load_subtree(): separate logic for internal vs. terminal entries load_subtree(): fix incorrect comment load_subtree(): reduce the scope of some local variables load_subtree(): remove unnecessary conditional notes: make GET_NIBBLE macro more robust

load_subtree(): check that `prefix_len` is in the expected range

2017-09-08T18:16:13Z

This value, which is stashed in the last byte of an object_id hash, gets handed around a lot. So add a sanity check before using it in `load_subtree()`. Signed-off-by: Michael Haggerty Signed-off-by: Junio C Hamano

load_subtree(): declare some variables to be `size_t`

2017-08-26T16:21:01Z

* `prefix_len` * `path_len` * `i` It's good hygiene. Signed-off-by: Michael Haggerty Signed-off-by: Junio C Hamano

hex_to_bytes(): simpler replacement for `get_oid_hex_segment()`

2017-08-26T16:21:01Z

Now that `get_oid_hex_segment()` does less, it makes sense to rename it and simplify its semantics: * Instead of a `hex_len` parameter, which was the number of hex characters (and had to be even), use a `len` parameter, which is the number of resulting bytes. This removes then need for the check that `hex_len` is even and to divide it by two to determine the number of bytes. For good hygiene, declare the `len` parameter to be `size_t` instead of `unsigned int`. * Change the order of the arguments to the more traditional (dst, src, len). * Rename the function to `hex_to_bytes()`. * Remove a loop variable: just count `len` down instead. Signed-off-by: Michael Haggerty Signed-off-by: Junio C Hamano

get_oid_hex_segment(): don't pad the rest of `oid`

2017-08-26T16:21:01Z

Remove the feature of `get_oid_hex_segment()` that it pads the rest of the `oid` argument with zeros. Instead, do this at the caller who needs it. This makes the functionality of this function more coherent and removes the need for its `oid_len` argument. Signed-off-by: Michael Haggerty Signed-off-by: Junio C Hamano

load_subtree(): combine some common code

2017-08-26T16:21:01Z

Write the length into `object_oid` (before copying) rather than `l->key_oid` (after copying). Then combine some code from the two `if` blocks. Signed-off-by: Michael Haggerty Signed-off-by: Junio C Hamano

get_oid_hex_segment(): return 0 on success

2017-08-26T16:21:01Z

Nobody cares about the return value of get_oid_hex_segment() except to check whether it failed. So just return 0 on success. And while we're updating its docstring, update it for some argument renaming that happened a while ago. Signed-off-by: Michael Haggerty Signed-off-by: Junio C Hamano