elextr commented on this pull request.
case SC_STATUS_BADALLOC:
+ sub_msg = "memory is exhausted"; + break; + case SC_STATUS_WARN_REGEX: + sub_msg = "regular expression is invalid"; + break; + default: + if (status >= SC_STATUS_WARN_START) + sub_msg = "unknown warning"; + else + sub_msg = "unknown failure"; + break; + } +#define SCI_STATUS_FORMAT_STRING "%s:%u: scintilla has non-zero status " \ + "code '%d' after sending message '%u' to instance '%p' with " \ + "wParam='%lu' and lParam='%ld': %s"
Theoretically these formats should be `%p` because both wparam and lparam are `intptr_t`. In reality they are the same size as long on all platforms except win64, and we don't currently support that AFAIK.