Make two 1-bit fields unsigned

This should be a completely harmless warning as we only check whether
the fields are true or false.

Closes #40911.

(Backported by nickm so that I can compile 0.4.8 without warnings.)

diff --git a/changes/bug40911 b/changes/bug40911
new file mode 100644 (file)
index 0000000..c938b56
--- /dev/null
+++ b/changes/bug40911
@@ -0,0 +1,5 @@
+  o Minor bugfixes (compiler warnings):
+    - Make sure the two bitfields in the half-closed edge struct are
+      unsigned, as we're using them for boolean values and assign 1 to
+      them. Fixes bug 40911; bugfix on 0.4.7.2-alpha.
+

diff --git a/src/core/or/half_edge_st.h b/src/core/or/half_edge_st.h
index 642d8e1ea5247bfdeac3b00ad96f2ca96906da11..d8c183a93c1f5086fc846860858f945ef8dfea1a 100644 (file)
--- a/src/core/or/half_edge_st.h
+++ b/src/core/or/half_edge_st.h
@@ -41,10 +41,10 @@ typedef struct half_edge_t {
   /**
    * Did this edge use congestion control? If so, use
    * timer instead of pending data approach */
-  int used_ccontrol : 1;
+  unsigned int used_ccontrol : 1;
 
   /** Is there a connected cell pending? */
-  int connected_pending : 1;
+  unsigned int connected_pending : 1;
 } half_edge_t;
 
 #endif /* !defined(HALF_EDGE_ST_H) */