Don't select the DataStream protocol by default for now
authorManuel Nickschas <sputnick@quassel-irc.org>
Sat, 15 Feb 2014 20:58:46 +0000 (21:58 +0100)
committerManuel Nickschas <sputnick@quassel-irc.org>
Sat, 15 Feb 2014 21:01:02 +0000 (22:01 +0100)
Until we have streaming compression implemented, using the DataStream
is not such a brilliant idea compared to the legacy protocol which
at least individually compresses messages. Thus, we should not select
the DataStream protocol by default.

Clients wanting to play around with the new protocol are still free to
negotiate this during handshake.

src/common/peerfactory.cpp

index 0dc0140..655a6e1 100644 (file)
@@ -27,8 +27,8 @@
 PeerFactory::ProtoList PeerFactory::supportedProtocols()
 {
     ProtoList result;
-    result.append(ProtoDescriptor(Protocol::DataStreamProtocol, DataStreamPeer::supportedFeatures()));
     result.append(ProtoDescriptor(Protocol::LegacyProtocol, 0));
+    result.append(ProtoDescriptor(Protocol::DataStreamProtocol, DataStreamPeer::supportedFeatures()));
     return result;
 }