Mir

Code review comment for lp:~afrantzis/mir/variable-length-serialization-buffers-1320187

Revision history for this message
Andreas Pokorny (andreas-pokorny) wrote :

> > What is the purpose of
> > connection_to_server_with_large_display_configuration_succeeds in
> > test_large_messages.cpp?
>
> Clarified purpose of test in new comment:
>
> 370 +// Regression test for lp:1320187. Without a fix for that bug,
> connecting
> 371 +// to a server with a lot of information in the display configuration
> hangs
> 372 +// the client, because the server reply message is
> incomplete/corrupted.

oh, ok.

review: Approve

« Back to merge proposal