diff options
| author | Pierre Ossman <ossman@cendio.se> | 2018-06-20 11:35:05 +0200 |
|---|---|---|
| committer | Pierre Ossman <ossman@cendio.se> | 2018-11-01 16:11:42 +0100 |
| commit | 96728356628e90ef80dc111269db1593be87aa16 (patch) | |
| tree | 05293401b2ab227be40d07a2e096d8e913d6cdea /common/rfb/SConnection.cxx | |
| parent | ef6881b9db3765ed07db9e1d414b178daae29606 (diff) | |
| download | tigervnc-96728356628e90ef80dc111269db1593be87aa16.tar.gz tigervnc-96728356628e90ef80dc111269db1593be87aa16.zip | |
Make arguments explicit in all message writer methods
Make sure all methods only write what is given as arguments, and
avoid side effects by getting data from parameter objects. This keeps
things readable in the calling code.
Diffstat (limited to 'common/rfb/SConnection.cxx')
| -rw-r--r-- | common/rfb/SConnection.cxx | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/common/rfb/SConnection.cxx b/common/rfb/SConnection.cxx index 52a1113d..d0490de5 100644 --- a/common/rfb/SConnection.cxx +++ b/common/rfb/SConnection.cxx @@ -355,7 +355,8 @@ void SConnection::approveConnection(bool accept, const char* reason) void SConnection::clientInit(bool shared) { - writer_->writeServerInit(); + writer_->writeServerInit(client.width(), client.height(), + client.pf(), client.name()); state_ = RFBSTATE_NORMAL; } |