Use 'unsigned:1' when we mean boolean options

These options are all strictly boolean (true/false).  Its easier to
document this implicitly by making their storage type a single bit.
There is no compelling memory space reduction reason for this change,
it just makes the structure definition slightly more readable.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

diff --git a/fetch-pack.h b/fetch-pack.h
index cdcd84f2b562d2f6a47a2ed2f4434034f639cbd5..ad1307689d2488011e99938b8091f3fd3428ccf8 100644 (file)
--- a/fetch-pack.h
+++ b/fetch-pack.h
@@ -4,14 +4,14 @@
 struct fetch_pack_args
 {
        const char *uploadpack;
-       int quiet;
-       int keep_pack;
        int unpacklimit;
-       int use_thin_pack;
-       int fetch_all;
-       int verbose;
        int depth;
-       int no_progress;
+       unsigned quiet:1,
+               keep_pack:1,
+               use_thin_pack:1,
+               fetch_all:1,
+               verbose:1,
+               no_progress:1;
 };
 
 void setup_fetch_pack(struct fetch_pack_args *args);