Apply suggestions from technical writer's review

Author: patiencedaur

doc/dev_guide/internals/iproto/events.rst

@@ -68,4 +68,4 @@ Sent by the server to notify a client about an update of a key.
     :file: images/event.svg
 
 ``IPROTO_EVENT_DATA`` contains data sent to a remote watcher.
-The parameter is optional, the default value is ``nil``.
+The parameter is optional, the default value is ``MP_NIL``.

doc/dev_guide/internals/iproto/format.rst

@@ -4,7 +4,7 @@ Request and response format
 ===========================
 
 The types referred to in this document are `MessagePack <http://MessagePack.org>`_ types.
-For type definitions, see the :ref:`Symbols and terms <box_protocol-notation>` section.
+For their definitions, see the :ref:`MP_* MessagePack types <box_protocol-notation>` section.
 
 ..  _internals-unified_packet_structure:
 
@@ -22,7 +22,7 @@ Size
 ----
 
 The size is an MP_UINT -- unsigned integer, usually 32-bit.
-It it the size of the header plus the size of the body.
+It the size of the header plus the size of the body.
 It may be useful to compare it with the number of bytes remaining in the packet.
 
 ..  _box_protocol-header:
@@ -65,7 +65,7 @@ the IPROTO_REQUEST_TYPE value will be 0 = ``IPROTO_OK`` and the
 array will have all the tuples of the result.
 
 Read the source code file `net_box.c <https://github.com/tarantool/tarantool/blob/master/src/box/lua/net_box.c>`__
-where the function "decode_metadata_optional" is an example of how Tarantool
+where the function ``decode_metadata_optional`` is an example of how Tarantool
 itself decodes extra items.
 
 Body

doc/dev_guide/internals/iproto/images/sql_response_other.puml

@@ -16,7 +16,8 @@ json "**Response to SQL requests other than SELECT, VALUES, or PRAGMA**" as sql_
     },
     "Body": {
       "[[tarantool.io/en/doc/latest/dev_guide/internals/iproto/keys SQL_INFO]]": {
-        "[[tarantool.io/en/doc/latest/dev_guide/internals/iproto/keys SQL_INFO_ROW_COUNT]]": "MP_UINT"
+        "[[tarantool.io/en/doc/latest/dev_guide/internals/iproto/keys SQL_INFO_ROW_COUNT]]": "MP_UINT",
+        "(Optional) [[tarantool.io/en/doc/latest/dev_guide/internals/iproto/keys SQL_INFO_AUTO_INCREMENT_IDS]]": "MP_ARRAY"
       }
     }
 }

doc/dev_guide/internals/iproto/images/sql_response_other.svg

@@ -40,7 +40,7 @@ General
 
         *   -   :ref:`IPROTO_SCHEMA_VERSION <internals-iproto-keys-schema_version>`
             -   0x05 |br|  MP_UINT
-            -   Identifier that is increased with every major change
+            -   Version of the database schema
 
         *   -   IPROTO_TIMESTAMP
             -   0x04 |br| MP_DOUBLE 
@@ -109,7 +109,7 @@ General
 
         *   -   IPROTO_EXPR
             -   0x27 |br| MP_STR
-            -   Command argument. Passed within :ref:`IPROTO_EVAL <box_protocol-eval>`
+            -   Command argument. Used in :ref:`IPROTO_EVAL <box_protocol-eval>`
 
 
 Streams
@@ -455,7 +455,8 @@ IPROTO_SCHEMA_VERSION
 
 Code: 0x05.
 
-An unsigned number that goes up when there is a major change in the schema.
+Version of the database schema --
+an unsigned number that goes up when there is a major change in the schema.
 
 In a *request* header, IPROTO_SCHEMA_VERSION is optional, so the version will not
 be checked if it is absent.

doc/dev_guide/internals/iproto/keys.rst

@@ -3,11 +3,14 @@
 MP_* MessagePack types
 ======================
 
-The binary protocol handles data in the MessagePack format, which makes use of MessagePack data types.
+The binary protocol handles data in the  `MessagePack <http://MessagePack.org>`_ format.
+Short descriptions of the basic MessagePack data types 
+are on MessagePack's `"spec" page <https://github.com/msgpack/msgpack/blob/master/spec.md>`_.
+Tarantool also introduces several MessagePack type :ref:`extensions <internals-msgpack_ext>`.
+
+In this document, MessagePack types are described by words that start with **MP_**.
+See this table:
 
-This document makes use of words that start with **MP_**. They mean
-a `MessagePack <http://MessagePack.org>`_ type or a range of MessagePack types,
-including the signal and possibly including a value, with slight modification:
 
 ..  container:: table
 
@@ -39,5 +42,3 @@ including the signal and possibly including a value, with slight modification:
             -   :ref:`extension <internals-msgpack_ext>`
         *   -   **MP_OBJECT**
             -   any MessagePack object
-
-Short descriptions are in MessagePack's `"spec" page <https://github.com/msgpack/msgpack/blob/master/spec.md>`_.

doc/dev_guide/internals/iproto/mp_types.rst

@@ -26,7 +26,7 @@ Basic request description
 
         *   -   :ref:`IPROTO_PREPARE <box_protocol-prepare>`
             -   0x0d
-            -   Prepare an SQL statement (:ref:`box.prepare <box-sql_box_prepare>`)
+            -   Prepare an SQL statement (:ref:`box.prepare() <box-sql_box_prepare>`)
 
 
 ..  _box_protocol-execute: