linker: have clang check the DL_ERROR_AFTER() format strings.
The other logging options are checked "for free" by virtue of all macro-expanding to an existing logging function that has the appropriate attribute. This was an exception, and I found and removed a harmless extra argument from one call in my previous change to this code. I expected more fallout from turning on the checking, so kept it for a separate patch, but it seems that the one mistake I already fixed was the only mistake.
Oh well, at least this will mean we can't regress...
Change-Id: I6f47ccf9c545ea5b74375f4a5ceb0951006b948e
diff --git a/linker/linker_globals.h b/linker/linker_globals.h
index e6b4bc6..777e7b8 100644
--- a/linker/linker_globals.h
+++ b/linker/linker_globals.h
@@ -51,7 +51,7 @@
async_safe_format_fd(2, "\n"); \
} while (false)
-bool DL_ERROR_AFTER(int target_sdk_version, const char* fmt, ...);
+bool DL_ERROR_AFTER(int target_sdk_version, const char* fmt, ...) __printflike(2, 3);
#define DL_ERR_AND_LOG(fmt, x...) \
do { \