From f91b91b6d8b5a45aef2b051a6b493500bf9f52dd Mon Sep 17 00:00:00 2001
From: Henry de Valence <hdevalence@hdevalence.ca>
Date: Tue, 1 Sep 2020 12:44:35 -0700
Subject: [PATCH] network: clarify comment on Default for handshake::Builder

Co-authored-by: Jane Lusby <jlusby42@gmail.com>
---
 zebra-network/src/peer/handshake.rs | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/zebra-network/src/peer/handshake.rs b/zebra-network/src/peer/handshake.rs
index 41b9a6757..77f881c8c 100644
--- a/zebra-network/src/peer/handshake.rs
+++ b/zebra-network/src/peer/handshake.rs
@@ -140,7 +140,9 @@ where
 {
     /// Create a builder that configures a [`Handshake`] service.
     pub fn builder() -> Builder<S> {
-        // can't derive Builder::default without a bound on S :(
+        // We don't derive `Default` because the derive inserts a `where S:
+        // Default` bound even though `Option<S>` implements `Default` even if
+        // `S` does not.
         Builder {
             config: None,
             inbound_service: None,