build: fix linking when used in several translation units

All non-template methods defined outside classes are not implicitly
`inline` - that violates one definition rule when the connector is
included in several translaction units. Let's mark such methods as
`inline` to fix this issue.

CMakeLists.txt

@@ -248,3 +248,8 @@ TNTCXX_TEST(NAME SchemalessExample TYPE other
             SOURCES examples/Schemaless.cpp
             LIBRARIES ${COMMON_LIB}
 )
+
+TNTCXX_TEST(NAME LinkTest TYPE other
+            SOURCES test/LinkTest/Main.cpp test/LinkTest/Secondary.cpp
+            LIBRARIES ${COMMON_LIB}
+)

src/Client/Scramble.hpp

@@ -36,7 +36,7 @@
 
 namespace tnt {
 
-tnt::Sha1_type
+inline tnt::Sha1_type
 scramble(std::string_view password, const char *salt)
 {
 	Sha1_type hash1 = sha1(password);

src/Client/Stream.hpp

@@ -181,7 +181,7 @@ class Stream {
 ////////////////////////// Implementation  //////////////////////////
 /////////////////////////////////////////////////////////////////////
 
-std::ostream &
+inline std::ostream &
 operator<<(std::ostream &strm, enum StreamTransport transport)
 {
 	if (transport == STREAM_PLAIN)
@@ -192,7 +192,7 @@ operator<<(std::ostream &strm, enum StreamTransport transport)
 		return strm << "unknown transport";
 }
 
-std::ostream &
+inline std::ostream &
 operator<<(std::ostream &strm, const ConnectOptions &opts)
 {
 	strm << opts.address;
@@ -203,7 +203,7 @@ operator<<(std::ostream &strm, const ConnectOptions &opts)
 	return strm;
 }
 
-int
+inline int
 Stream::set_status(uint32_t st)
 {
 	if (st & SS_READINESS_STATUS)
@@ -219,7 +219,7 @@ Stream::set_status(uint32_t st)
 	return 0;
 }
 
-int
+inline int
 Stream::remove_status(uint32_t st)
 {
 	assert(!(st & SS_READINESS_STATUS));

src/Client/UnixPlainStream.hpp

@@ -71,7 +71,7 @@ class UnixPlainStream : public UnixStream {
 ////////////////////////// Implementation  //////////////////////////
 /////////////////////////////////////////////////////////////////////
 
-int
+inline int
 UnixPlainStream::connect(const ConnectOptions &opts)
 {
 	if (opts.transport != STREAM_PLAIN)
@@ -81,7 +81,7 @@ UnixPlainStream::connect(const ConnectOptions &opts)
 }
 
 namespace internal {
-struct msghdr
+inline struct msghdr
 create_msghdr(struct iovec *iov, size_t iov_count)
 {
 	struct msghdr msg{};
@@ -91,7 +91,7 @@ create_msghdr(struct iovec *iov, size_t iov_count)
 }
 } // namespace internal
 
-ssize_t
+inline ssize_t
 UnixPlainStream::send(struct iovec *iov, size_t iov_count)
 {
 	if (!(has_status(SS_ESTABLISHED))) {
@@ -120,7 +120,7 @@ UnixPlainStream::send(struct iovec *iov, size_t iov_count)
 	}
 }
 
-ssize_t
+inline ssize_t
 UnixPlainStream::recv(struct iovec *iov, size_t iov_count)
 {
 	if (!(has_status(SS_ESTABLISHED))) {

src/Client/UnixStream.hpp

@@ -142,7 +142,7 @@ UnixStream::tell(StreamStatus st, const char *file, int line,
 	return 0;
 }
 
-int
+inline int
 UnixStream::check_pending()
 {
 	assert(has_status(SS_CONNECT_PENDING));
@@ -154,7 +154,7 @@ UnixStream::check_pending()
 	return US_DIE("Failed to connect", strerror(rc == 0 ? err : errno));
 }
 
-int
+inline int
 UnixStream::connect(const ConnectOptions &opts_arg)
 {
 	if (!has_status(SS_DEAD))
@@ -217,12 +217,14 @@ UnixStream::connect(const ConnectOptions &opts_arg)
 		return US_DIE("Failed to connect");
 }
 
+inline
 UnixStream::~UnixStream() noexcept
 {
 	close();
 }
 
-void UnixStream::close()
+inline void
+UnixStream::close()
 {
 	if (fd >= 0) {
 		if (::close(fd) == 0)

src/Utils/AddrInfo.hpp

@@ -91,6 +91,7 @@ AddrInfo::~AddrInfo() noexcept
 		freeaddrinfo(infos);
 }
 
+inline
 AddrInfo::AddrInfo(const std::string &addr, const std::string &service)
 {
 	resolve(addr, service);

src/Utils/Sha1.hpp

@@ -71,6 +71,7 @@ inline void sha1_xor(Sha1_type &a, const Sha1_type &b);
 ////////////////////////// Implementation  //////////////////////////
 /////////////////////////////////////////////////////////////////////
 
+inline
 Sha1Calc::Sha1Calc()
 {
 	SHA1Init(&ctx);