This project is archived. Its data is read-only. This project is read-only.
Templated constructor of String should be deleted
The following discussions from !10 should be addressed: - [x] @klinkovsky started a [discussion](https://mmg-gitlab.fjfi.cvut.cz/gitlab/tnl/tnl-dev/merge_requests/10#note_124): (+1 comment) > This should be a free function, not a constructor (note that there is already `convertToString` which is overloaded for `bool`). At the very least the constructor should be explicit. - [x] @klinkovsky started a [discussion](https://mmg-gitlab.fjfi.cvut.cz/gitlab/tnl/tnl-dev/merge_requests/10#note_125): (+1 comment) > The recursion should be avoided by deleting the templated constructor and using the free function `convertToString` explicitly instead. It can be renamed to `toString` or something like that to make it shorter, c.f. [std::to_string](https://en.cppreference.com/w/cpp/string/basic_string/to_string).
issue