search

f1nal3 / Juniorgram

A C++ messenger for learning purposes in EPAM Brest C++ R&D Lab
14 stars 1 forks source link

Common changes for Client && ConnectionManager #619

Closed memorisecodead closed 2 years ago

memorisecodead commented 2 years ago

Description:

For our vision, we should fixing some code smells here. At work with tests, we struggle with some problems with override functions and incorrect signature here. In order to correctly complete the work on the tests, it is necessary to correct some points.

Acceptance Criteria:

  1. Fixing code smells.
  2. Does try signature functions for override in future.
  3. Small Refactoring for documentation.