Feature Request: verbose&notice&warning&error&fatalError ---addColor..

[yty]

I think it should be better to differentiate log files.
How do you think about it?

verbose->blur
notice->green
warning->Orange
error ->darkred
fatalError ->red

string ofGetLogLevelName(ofLogLevel level){
    HANDLE hCout = GetStdHandle(STD_OUTPUT_HANDLE); 

    switch(level){
        case OF_LOG_VERBOSE:
            SetConsoleTextAttribute(hCout,FOREGROUND_BLUE );
            return "verbose";
            break;
        case OF_LOG_NOTICE:
            SetConsoleTextAttribute(hCout,FOREGROUND_GREEN );
            return "notice";
            break;
        case OF_LOG_WARNING:
            SetConsoleTextAttribute(hCout,FOREGROUND_RED|FOREGROUND_GREEN );
            return "warning";
            break;
        case OF_LOG_ERROR:
            SetConsoleTextAttribute(hCout,FOREGROUND_RED );
            return "error";
            break;
        case OF_LOG_FATAL_ERROR:
            SetConsoleTextAttribute(hCout,FOREGROUND_RED|FOREGROUND_INTENSITY );
            return "fatal error";
            break;
        case OF_LOG_SILENT:
            return "silent";
            break;
        default:
            return "";
    }
}

void ofConsoleLoggerChannel::log(ofLogLevel level, const string & module, const string & message){
    // print to cerr for OF_LOG_ERROR and OF_LOG_FATAL_ERROR, everything else to cout 
    ostream& out = level < OF_LOG_ERROR ? cout : cerr;
    out << "[";
    // only print the module name if it's not "OF"
    if(module != "OF") {
        out << module << ":";
    }
    HANDLE hCout = GetStdHandle(STD_OUTPUT_HANDLE); 

    out << ofGetLogLevelName(level);
    SetConsoleTextAttribute(hCout,FOREGROUND_RED|FOREGROUND_GREEN|FOREGROUND_BLUE );
    out<< "] "<< message << endl;
}   

[bilderbuchi]

have you tried the logging system very recently? we improved the layout so that logging messages can be distinguished better, check it out on develop!

[yty]

@ bilderbuchi

Adding color is not good?

[bilderbuchi]

not if it's not really necessary in the first place, gets lost when saving logging to file anyway, is a platform-specific implementation.

[Teatoller]

The issue at hand here was one of striking a consensus. That is on the use of color styling to differentiate for verbose, notice, warnings, and error, or fatal error messages.
It is NOT a bug.
There has been no conclusive decision on this for close to 8 years.
May we close this issue and reopen it when needed.