Core/Log: Pass string_view by value instead of ref
This commit is contained in:
parent
e96204a2ab
commit
b38f9dc79b
|
|
@ -26,8 +26,8 @@ namespace Nz
|
||||||
|
|
||||||
virtual bool IsStdReplicationEnabled() const = 0;
|
virtual bool IsStdReplicationEnabled() const = 0;
|
||||||
|
|
||||||
virtual void Write(const std::string_view& string) = 0;
|
virtual void Write(std::string_view string) = 0;
|
||||||
virtual void WriteError(ErrorType type, const std::string_view& error, unsigned int line = 0, const char* file = nullptr, const char* function = nullptr);
|
virtual void WriteError(ErrorType type, std::string_view error, unsigned int line = 0, const char* file = nullptr, const char* function = nullptr);
|
||||||
|
|
||||||
AbstractLogger& operator=(const AbstractLogger&) = default;
|
AbstractLogger& operator=(const AbstractLogger&) = default;
|
||||||
AbstractLogger& operator=(AbstractLogger&&) noexcept = default;
|
AbstractLogger& operator=(AbstractLogger&&) noexcept = default;
|
||||||
|
|
|
||||||
|
|
@ -29,8 +29,8 @@ namespace Nz
|
||||||
bool IsStdReplicationEnabled() const override;
|
bool IsStdReplicationEnabled() const override;
|
||||||
bool IsTimeLoggingEnabled() const;
|
bool IsTimeLoggingEnabled() const;
|
||||||
|
|
||||||
void Write(const std::string_view& string) override;
|
void Write(std::string_view string) override;
|
||||||
void WriteError(ErrorType type, const std::string_view& error, unsigned int line = 0, const char* file = nullptr, const char* function = nullptr) override;
|
void WriteError(ErrorType type, std::string_view error, unsigned int line = 0, const char* file = nullptr, const char* function = nullptr) override;
|
||||||
|
|
||||||
FileLogger& operator=(const FileLogger&) = delete;
|
FileLogger& operator=(const FileLogger&) = delete;
|
||||||
FileLogger& operator=(FileLogger&&) = default;
|
FileLogger& operator=(FileLogger&&) = default;
|
||||||
|
|
|
||||||
|
|
@ -39,8 +39,8 @@ namespace Nz
|
||||||
|
|
||||||
static void SetLogger(AbstractLogger* logger);
|
static void SetLogger(AbstractLogger* logger);
|
||||||
|
|
||||||
static void Write(const std::string_view& string);
|
static void Write(std::string_view string);
|
||||||
static void WriteError(ErrorType type, const std::string_view& error, unsigned int line = 0, const char* file = nullptr, const char* function = nullptr);
|
static void WriteError(ErrorType type, std::string_view error, unsigned int line = 0, const char* file = nullptr, const char* function = nullptr);
|
||||||
|
|
||||||
NazaraStaticSignal(OnLogWrite, const std::string_view& /*string*/);
|
NazaraStaticSignal(OnLogWrite, const std::string_view& /*string*/);
|
||||||
NazaraStaticSignal(OnLogWriteError, ErrorType /*type*/, const std::string_view& /*error*/, unsigned int /*line*/, const char* /*file*/, const char* /*function*/);
|
NazaraStaticSignal(OnLogWriteError, ErrorType /*type*/, const std::string_view& /*error*/, unsigned int /*line*/, const char* /*file*/, const char* /*function*/);
|
||||||
|
|
|
||||||
|
|
@ -24,8 +24,8 @@ namespace Nz
|
||||||
|
|
||||||
bool IsStdReplicationEnabled() const override;
|
bool IsStdReplicationEnabled() const override;
|
||||||
|
|
||||||
void Write(const std::string_view& error) override;
|
void Write(std::string_view string) override;
|
||||||
void WriteError(ErrorType type, const std::string_view& error, unsigned int line = 0, const char* file = nullptr, const char* function = nullptr) override;
|
void WriteError(ErrorType type, std::string_view error, unsigned int line = 0, const char* file = nullptr, const char* function = nullptr) override;
|
||||||
|
|
||||||
StdLogger& operator=(const StdLogger&) = default;
|
StdLogger& operator=(const StdLogger&) = default;
|
||||||
StdLogger& operator=(StdLogger&&) noexcept = default;
|
StdLogger& operator=(StdLogger&&) noexcept = default;
|
||||||
|
|
|
||||||
|
|
@ -53,7 +53,7 @@ namespace Nz
|
||||||
bool SetCursorPos(UInt64 offset);
|
bool SetCursorPos(UInt64 offset);
|
||||||
|
|
||||||
bool Write(const ByteArray& byteArray);
|
bool Write(const ByteArray& byteArray);
|
||||||
bool Write(const std::string_view& string);
|
bool Write(std::string_view string);
|
||||||
inline std::size_t Write(const void* buffer, std::size_t size);
|
inline std::size_t Write(const void* buffer, std::size_t size);
|
||||||
|
|
||||||
Stream& operator=(const Stream&) = delete;
|
Stream& operator=(const Stream&) = delete;
|
||||||
|
|
|
||||||
|
|
@ -39,7 +39,7 @@ namespace Nz
|
||||||
* \param file Filename
|
* \param file Filename
|
||||||
* \param function Name of the function throwing the error
|
* \param function Name of the function throwing the error
|
||||||
*/
|
*/
|
||||||
void AbstractLogger::WriteError(ErrorType type, const std::string_view& error, unsigned int line, const char* file, const char* function)
|
void AbstractLogger::WriteError(ErrorType type, std::string_view error, unsigned int line, const char* file, const char* function)
|
||||||
{
|
{
|
||||||
NAZARA_USE_ANONYMOUS_NAMESPACE
|
NAZARA_USE_ANONYMOUS_NAMESPACE
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -91,7 +91,7 @@ namespace Nz
|
||||||
* \see WriteError
|
* \see WriteError
|
||||||
*/
|
*/
|
||||||
|
|
||||||
void FileLogger::Write(const std::string_view& string)
|
void FileLogger::Write(std::string_view string)
|
||||||
{
|
{
|
||||||
if (m_forceStdOutput || m_stdReplicationEnabled)
|
if (m_forceStdOutput || m_stdReplicationEnabled)
|
||||||
{
|
{
|
||||||
|
|
@ -147,7 +147,7 @@ namespace Nz
|
||||||
* \see Write
|
* \see Write
|
||||||
*/
|
*/
|
||||||
|
|
||||||
void FileLogger::WriteError(ErrorType type, const std::string_view& error, unsigned int line, const char* file, const char* function)
|
void FileLogger::WriteError(ErrorType type, std::string_view error, unsigned int line, const char* file, const char* function)
|
||||||
{
|
{
|
||||||
AbstractLogger::WriteError(type, error, line, file, function);
|
AbstractLogger::WriteError(type, error, line, file, function);
|
||||||
m_outputFile.flush();
|
m_outputFile.flush();
|
||||||
|
|
|
||||||
|
|
@ -78,7 +78,7 @@ namespace Nz
|
||||||
* \see WriteError
|
* \see WriteError
|
||||||
*/
|
*/
|
||||||
|
|
||||||
void Log::Write(const std::string_view& string)
|
void Log::Write(std::string_view string)
|
||||||
{
|
{
|
||||||
if (s_enabled)
|
if (s_enabled)
|
||||||
s_logger->Write(string);
|
s_logger->Write(string);
|
||||||
|
|
@ -98,7 +98,7 @@ namespace Nz
|
||||||
* \see Write
|
* \see Write
|
||||||
*/
|
*/
|
||||||
|
|
||||||
void Log::WriteError(ErrorType type, const std::string_view& error, unsigned int line, const char* file, const char* function)
|
void Log::WriteError(ErrorType type, std::string_view error, unsigned int line, const char* file, const char* function)
|
||||||
{
|
{
|
||||||
if (s_enabled)
|
if (s_enabled)
|
||||||
s_logger->WriteError(type, error, line, file, function);
|
s_logger->WriteError(type, error, line, file, function);
|
||||||
|
|
|
||||||
|
|
@ -61,9 +61,9 @@ namespace Nz
|
||||||
* \see WriteError
|
* \see WriteError
|
||||||
*/
|
*/
|
||||||
|
|
||||||
void StdLogger::Write(const std::string_view& error)
|
void StdLogger::Write(std::string_view string)
|
||||||
{
|
{
|
||||||
fwrite(error.data(), sizeof(char), error.size(), stdout);
|
fwrite(string.data(), sizeof(char), string.size(), stdout);
|
||||||
fputc('\n', stdout);
|
fputc('\n', stdout);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
@ -79,7 +79,7 @@ namespace Nz
|
||||||
* \see Write
|
* \see Write
|
||||||
*/
|
*/
|
||||||
|
|
||||||
void StdLogger::WriteError(ErrorType type, const std::string_view& error, unsigned int line, const char* file, const char* function)
|
void StdLogger::WriteError(ErrorType type, std::string_view error, unsigned int line, const char* file, const char* function)
|
||||||
{
|
{
|
||||||
NAZARA_USE_ANONYMOUS_NAMESPACE
|
NAZARA_USE_ANONYMOUS_NAMESPACE
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -257,7 +257,7 @@ namespace Nz
|
||||||
* \param string String to write
|
* \param string String to write
|
||||||
*/
|
*/
|
||||||
|
|
||||||
bool Stream::Write(const std::string_view& string)
|
bool Stream::Write(std::string_view string)
|
||||||
{
|
{
|
||||||
if (m_streamOptions & StreamOption::Text)
|
if (m_streamOptions & StreamOption::Text)
|
||||||
{
|
{
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue