Repository: gabime/spdlog
File: include/spdlog/sinks/android_sink.h (C/C++)

Code before:
protected:
    void _sink_it(const details::log_msg& msg) override
    {
        const android_LogPriority priority = convert_to_android(msg.level);
        const int expected_size = msg.formatted.size();
        const int size = __android_log_write(
            priority, _tag.c_str(), msg.formatted.c_str()
        );
        if (size > expected_size)
        {
            // Will write a little bit more than original message
        }
        else
        {
            throw spdlog_ex("Send to Android logcat failed");
        }
    }

private:
    static android_LogPriority convert_to_android(spdlog::level::level_enum level)
    {
        switch(level)
        {
            case spdlog::level::trace: return ANDROID_LOG_VERBOSE;
            case spdlog::level::debug: return ANDROID_LOG_DEBUG;
            case spdlog::level::info: return ANDROID_LOG_INFO;
            case spdlog::level::notice: return ANDROID_LOG_INFO;
            case spdlog::level::warn: return ANDROID_LOG_WARN;
            case spdlog::level::err: return ANDROID_LOG_ERROR;
            case spdlog::level::critical: return ANDROID_LOG_FATAL;
            case spdlog::level::alert: return ANDROID_LOG_FATAL;
            case spdlog::level::emerg: return ANDROID_LOG_FATAL;
            case spdlog::level::off: throw spdlog_ex("Unexpected off");
            default: throw spdlog_ex("Incorrect level value");
        }
    }

    std::string _tag;
};

typedef base_android_sink<std::mutex> android_sink_mt;
typedef base_android_sink<details::null_mutex> android_sink_st;

}
}

#endif


Review comment (performance):
@gabime I've wrote a [test](https://github.com/ruslo/spdlog/blob/android.test/example/android.cpp) with [two variants of converting](https://github.com/ruslo/spdlog/blob/android.test/include/spdlog/sinks/android_sink.h). Here is result:

```
Test:switch time:11258605569
Test:array time:10944504215
Test:switch time:11012342079
Test:array time:11089379475
Test:switch time:10916645205
Test:array time:10934150672
```

I see no significant performance difference even in this syntetic test. Actually in two last tests "switch" is faster.


Code after:
protected:
    void _sink_it(const details::log_msg& msg) override
    {
        if (msg.level == spdlog::level::off)
        {
            return;
        }
        const android_LogPriority priority = convert_to_android(msg.level);
        const int expected_size = msg.formatted.size();
        const int size = __android_log_write(
            priority, _tag.c_str(), msg.formatted.c_str()
        );
        if (size > expected_size)
        {
            // Will write a little bit more than original message
        }
        else
        {
            throw spdlog_ex("Send to Android logcat failed");
        }
    }

private:
    static android_LogPriority convert_to_android(spdlog::level::level_enum level)
    {
        switch(level)
        {
            case spdlog::level::trace: return ANDROID_LOG_VERBOSE;
            case spdlog::level::debug: return ANDROID_LOG_DEBUG;
            case spdlog::level::info: return ANDROID_LOG_INFO;
            case spdlog::level::notice: return ANDROID_LOG_INFO;
            case spdlog::level::warn: return ANDROID_LOG_WARN;
            case spdlog::level::err: return ANDROID_LOG_ERROR;
            case spdlog::level::critical: return ANDROID_LOG_FATAL;
            case spdlog::level::alert: return ANDROID_LOG_FATAL;
            case spdlog::level::emerg: return ANDROID_LOG_FATAL;
            default: throw spdlog_ex("Incorrect level value");
        }
    }

    std::string _tag;
};

typedef base_android_sink<std::mutex> android_sink_mt;
typedef base_android_sink<details::null_mutex> android_sink_st;

}
}

#endif