This is an automated email from Gerrit. Antonio Borneo ([email protected]) just uploaded a new patch set to Gerrit, which you can find at http://openocd.zylin.com/5008
-- gerrit commit 5f9bdcc8b5754d642e0fa3bedb075413d78a5258 Author: Antonio Borneo <[email protected]> Date: Sun Mar 31 09:50:38 2019 +0200 COVER_LETTER: usage field From commit 54d6330b78f4 (back to January 2012) we get debug messages about empty usage field: BUG: command '%s' does not have the '.usage' field filled out The idea of the developer was that people will start sending patches to fill the usage fields. This append only partially. This patch set: - cleanup the syntax while init struct command_registration, so simple scripts can be used to detect the wrong cases; - detects all the places where usage is not initialized, check the associated command and propose either an empty usage (for commands that do not require parameters) or a reasonable usage text; - change the behaviour of commands add_help_text/add_usage_text so they do not trigger either the BUG above nor other annoying messages; - replace the LOG_DEBUG with a LOG_ERROR, so the BUG message above gets much more visible. It will not show off with the current code, but will prevent developers to commit new commands with still empty usage field. Change-Id: I3178cf8b375ff8d67b1fc9f39e181d06622a06d3 Signed-off-by: Antonio Borneo <[email protected]> -- _______________________________________________ OpenOCD-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openocd-devel
