read_raw_ref(): change flags parameter to unsigned int

read_raw_ref() is going to be part of the vtable for reference backends, so clean up its interface to use "unsigned int flags" rather than "int flags". Its caller still uses signed int for its flags arguments. But changing that would touch a lot of code, so leave it for now. Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu> Signed-off-by: David Turner <dturner@twopensource.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>

Michael Haggerty committed Apr 7, 2016 at 15:03 UTC 89e82389655e3792bebad311d74541c3c0c6cc86
1 file changed +3 -3
refs/files-backend.c
+3 -3
@@ -1372,7 +1372,7 @@ static struct ref_entry *get_packed_ref(const char *refname)
1372 */
1373 static int resolve_missing_loose_ref(const char *refname,
1374 unsigned char *sha1,
1375 - int *flags)
1375 + unsigned int *flags)
1376 {
1377 struct ref_entry *entry;
1378
@@ -1421,7 +1421,7 @@ static int resolve_missing_loose_ref(const char *refname,
1421 * refname will still be valid and unchanged.
1422 */
1423 static int read_raw_ref(const char *refname, unsigned char *sha1,
1424 - struct strbuf *symref, int *flags)
1424 + struct strbuf *symref, unsigned int *flags)
1425 {
1426 struct strbuf sb_contents = STRBUF_INIT;
1427 struct strbuf sb_path = STRBUF_INIT;
@@ -1570,7 +1570,7 @@ const char *resolve_ref_unsafe(const char *refname, int resolve_flags,
1570 }
1571
1572 for (symref_count = 0; symref_count < MAXDEPTH; symref_count++) {
1573 - int read_flags = 0;
1573 + unsigned int read_flags = 0;
1574
1575 if (read_raw_ref(refname, sha1, &sb_refname, &read_flags)) {
1576 *flags |= read_flags;