Re: [PATCH 1/1] Fix some typos

[Laurent Vivier <laurent@vivier.eu>]

Le 30/11/2022 à 09:29, Philippe Mathieu-Daudé a écrit :
> On 30/11/22 02:53, Dongdong Zhang wrote:
>> Fix some typos in 'python' directory.
>>
>> Signed-off-by: Dongdong Zhang <zhangdongdong@eswincomputing.com>
> 
> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> 
>> ---
>>   python/qemu/machine/console_socket.py | 2 +-
>>   python/qemu/machine/qtest.py          | 2 +-
>>   python/qemu/qmp/protocol.py           | 2 +-
>>   python/qemu/qmp/qmp_tui.py            | 6 +++---
>>   4 files changed, 6 insertions(+), 6 deletions(-)
>>
>> diff --git a/python/qemu/machine/console_socket.py b/python/qemu/machine/console_socket.py
>> index 8c4ff598ad..4e28ba9bb2 100644
>> --- a/python/qemu/machine/console_socket.py
>> +++ b/python/qemu/machine/console_socket.py
>> @@ -68,7 +68,7 @@ def _thread_start(self) -> threading.Thread:
>>           """Kick off a thread to drain the socket."""
>>           # Configure socket to not block and timeout.
>>           # This allows our drain thread to not block
>> -        # on recieve and exit smoothly.
>> +        # on receive and exit smoothly.
>>           socket.socket.setblocking(self, False)
>>           socket.socket.settimeout(self, 1)
>>           drain_thread = threading.Thread(target=self._drain_fn)
>> diff --git a/python/qemu/machine/qtest.py b/python/qemu/machine/qtest.py
>> index 1a1fc6c9b0..906bd13298 100644
>> --- a/python/qemu/machine/qtest.py
>> +++ b/python/qemu/machine/qtest.py
>> @@ -42,7 +42,7 @@ class QEMUQtestProtocol:
>>       :raise socket.error: on socket connection errors
>>       .. note::
>> -       No conection is estabalished by __init__(), this is done
>> +       No connection is estabalished by __init__(), this is done
>>          by the connect() or accept() methods.
>>       """
>>       def __init__(self, address: SocketAddrT,
>> diff --git a/python/qemu/qmp/protocol.py b/python/qemu/qmp/protocol.py
>> index 6ea86650ad..15909b7dba 100644
>> --- a/python/qemu/qmp/protocol.py
>> +++ b/python/qemu/qmp/protocol.py
>> @@ -812,7 +812,7 @@ def _done(task: Optional['asyncio.Future[Any]']) -> bool:
>>       @bottom_half
>>       async def _bh_close_stream(self, error_pathway: bool = False) -> None:
>> -        # NB: Closing the writer also implcitly closes the reader.
>> +        # NB: Closing the writer also implicitly closes the reader.
>>           if not self._writer:
>>               return
>> diff --git a/python/qemu/qmp/qmp_tui.py b/python/qemu/qmp/qmp_tui.py
>> index ce239d8979..8369144723 100644
>> --- a/python/qemu/qmp/qmp_tui.py
>> +++ b/python/qemu/qmp/qmp_tui.py
>> @@ -71,7 +71,7 @@ def format_json(msg: str) -> str:
>>       due to an decoding error then a simple string manipulation is done to
>>       achieve a single line JSON string.
>> -    Converting into single line is more asthetically pleasing when looking
>> +    Converting into single line is more aesthetically pleasing when looking
>>       along with error messages.
>>       Eg:
>> @@ -91,7 +91,7 @@ def format_json(msg: str) -> str:
>>           [1, true, 3]: QMP message is not a JSON object.
>> -    The single line mode is more asthetically pleasing.
>> +    The single line mode is more aesthetically pleasing.
>>       :param msg:
>>           The message to formatted into single line.
>> @@ -498,7 +498,7 @@ def __init__(self, parent: App) -> None:
>>   class HistoryBox(urwid.ListBox):
>>       """
>>       This widget is modelled using the ListBox widget, contains the list of
>> -    all messages both QMP messages and log messsages to be shown in the TUI.
>> +    all messages both QMP messages and log messages to be shown in the TUI.
>>       The messages are urwid.Text widgets. On every append of a message, the
>>       focus is shifted to the last appended message.
> 
> 

Applied to my trivial-patches branch.

Thanks,
Laurent