<p><b>@jcarrano</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/RIOT-OS/RIOT/pull/11573#discussion_r287798490">sys/log/log_color/log_module.h</a>:</p>
<pre style='color:#555'>> +    va_start(args, format);
+    /* Temporarily disable clang format-nonliteral warning */
+#ifdef __clang__
+#pragma clang diagnostic push
+#pragma clang diagnostic ignored "-Wformat-nonliteral"
+#endif /* clang */
+    vprintf(format, args);
+#ifdef __clang__
+#pragma clang diagnostic pop
+#endif /* clang */
+    va_end(args);
+    printf(LOG_RESET_ANSI_COLOR_CODE);
+
+#ifdef MODULE_NEWLIB
+    /* no fflush on msp430 */
+    fflush(stdout);
</pre>
<p>Given that the terminal or the riot app can terminate at any moment (and that there is no flush in MSP430), one cannot really count on the RIOT app to reset the terminal state. The robust solution is to have "make term" or pyterm handle this by resetting after exit.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/RIOT-OS/RIOT/pull/11573?email_source=notifications&email_token=ABE7WYBK72URTBOBI7Y53ULPXPRM7A5CNFSM4HPOZHPKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOBZYQDLQ#discussion_r287798490">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABE7WYFHBZPB4VMJMUUD3QDPXPRM7ANCNFSM4HPOZHPA">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABE7WYHIAM74JDNXXDYO6S3PXPRM7A5CNFSM4HPOZHPKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOBZYQDLQ.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/RIOT-OS/RIOT/pull/11573?email_source=notifications\u0026email_token=ABE7WYBK72URTBOBI7Y53ULPXPRM7A5CNFSM4HPOZHPKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOBZYQDLQ#discussion_r287798490",
"url": "https://github.com/RIOT-OS/RIOT/pull/11573?email_source=notifications\u0026email_token=ABE7WYBK72URTBOBI7Y53ULPXPRM7A5CNFSM4HPOZHPKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOBZYQDLQ#discussion_r287798490",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>