From ecef8f93b5cfe8b92bf075131d40aca4e681a568 Mon Sep 17 00:00:00 2001
From: Jonathan Warren <me@jonwarren.org>
Date: Wed, 7 Aug 2013 15:15:49 -0400
Subject: [PATCH] minor code refactoring

---
 src/shared.py | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/shared.py b/src/shared.py
index 925459f7..09a2c26c 100644
--- a/src/shared.py
+++ b/src/shared.py
@@ -101,10 +101,8 @@ def assembleVersionMessage(remoteHost, remotePort, myStreamNumber):
 
     random.seed()
     payload += eightBytesOfRandomDataUsedToDetectConnectionsToSelf
-    userAgent = '/PyBitmessage:' + shared.softwareVersion + \
-        '/'  # Length of userAgent must be less than 253.
-    payload += pack('>B', len(
-        userAgent))  # user agent string length. If the user agent is more than 252 bytes long, this code isn't going to work.
+    userAgent = '/PyBitmessage:' + shared.softwareVersion + '/'
+    payload += encodeVarint(len(userAgent))
     payload += userAgent
     payload += encodeVarint(
         1)  # The number of streams about which I care. PyBitmessage currently only supports 1 per connection.