Patchwork D3383: wireprotov2: change frame type value for command data

login
register
mail settings
Submitter phabricator
Date April 16, 2018, 11:10 p.m.
Message ID <29df16442258f48898363ebb67a98a0c@localhost.localdomain>
Download mbox | patch
Permalink /patch/31114/
State Not Applicable
Headers show

Comments

phabricator - April 16, 2018, 11:10 p.m.
This revision was automatically updated to reflect the committed changes.
Closed by commit rHGe8fba6d578f0: wireprotov2: change frame type value for command data (authored by indygreg, committed by ).

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D3383?vs=8295&id=8326

REVISION DETAIL
  https://phab.mercurial-scm.org/D3383

AFFECTED FILES
  mercurial/help/internals/wireprotocol.txt
  mercurial/wireprotoframing.py
  tests/test-wireproto-serverreactor.py

CHANGE DETAILS




To: indygreg, #hg-reviewers, durin42
Cc: mercurial-devel

Patch

diff --git a/tests/test-wireproto-serverreactor.py b/tests/test-wireproto-serverreactor.py
--- a/tests/test-wireproto-serverreactor.py
+++ b/tests/test-wireproto-serverreactor.py
@@ -185,7 +185,7 @@ 
             ffs(b'1 1 stream-begin command-data 0 ignored'))
         self.assertaction(result, b'error')
         self.assertEqual(result[1], {
-            b'message': b'expected command request frame; got 3',
+            b'message': b'expected command request frame; got 2',
         })
 
     def testunexpectedcommanddatareceiving(self):
diff --git a/mercurial/wireprotoframing.py b/mercurial/wireprotoframing.py
--- a/mercurial/wireprotoframing.py
+++ b/mercurial/wireprotoframing.py
@@ -42,7 +42,7 @@ 
 }
 
 FRAME_TYPE_COMMAND_REQUEST = 0x01
-FRAME_TYPE_COMMAND_DATA = 0x03
+FRAME_TYPE_COMMAND_DATA = 0x02
 FRAME_TYPE_BYTES_RESPONSE = 0x04
 FRAME_TYPE_ERROR_RESPONSE = 0x05
 FRAME_TYPE_TEXT_OUTPUT = 0x06
diff --git a/mercurial/help/internals/wireprotocol.txt b/mercurial/help/internals/wireprotocol.txt
--- a/mercurial/help/internals/wireprotocol.txt
+++ b/mercurial/help/internals/wireprotocol.txt
@@ -638,10 +638,10 @@ 
 ``0x01`` or ``0x02`` MUST be set to indicate this frame's role in
 a series of command request frames.
 
-If command data frames are to be sent, ``0x10`` MUST be set on ALL
+If command data frames are to be sent, ``0x08`` MUST be set on ALL
 command request frames.
 
-Command Data (``0x03``)
+Command Data (``0x02``)
 -----------------------
 
 This frame contains raw data for a command.