simplify compare_cached_resolves_by_expiry() to make it match

the idioms of other compare functions. hopefully i didn't
break it?


svn:r7048

src/or/connection_edge.c

@@ -321,7 +321,7 @@ connection_edge_finished_connecting(edge_connection_t *edge_conn)
  * application connections. Rather than waiting a fixed amount of
  * time between each retry, we wait only 5 seconds for the first,
  * 10 seconds for the second, and 15 seconds for each retry after
- * that. Hopefully this will improve the expected experience. */
+ * that. Hopefully this will improve the expected user experience. */
 static int
 compute_socks_timeout(edge_connection_t *conn)
 {

src/or/dns.c

@@ -208,12 +208,7 @@ static int
 _compare_cached_resolves_by_expiry(const void *_a, const void *_b)
 {
   const cached_resolve_t *a = _a, *b = _b;
-  if (a->expire < b->expire)
-    return -1;
-  else if (a->expire == b->expire)
-    return 0;
-  else
-    return 1;
+  return a->expire - b->expire;
 }
 
 /** Priority queue of cached_resolve_t objects to let us know when they