Skip to content

Commit

Permalink
merge bitcoin#24849: Convert lint-logs.sh to Python
Browse files Browse the repository at this point in the history
  • Loading branch information
kwvg committed Nov 27, 2024
1 parent dbfd0b0 commit fe16516
Show file tree
Hide file tree
Showing 3 changed files with 35 additions and 30 deletions.
2 changes: 1 addition & 1 deletion src/random.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
#include <compat.h> // for Windows API
#include <wincrypt.h>
#endif
#include <logging.h> // for LogPrintf()
#include <logging.h>
#include <randomenv.h>
#include <support/allocators/secure.h>
#include <span.h>
Expand Down
34 changes: 34 additions & 0 deletions test/lint/lint-logs.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
#!/usr/bin/env python3
#
# Copyright (c) 2018-2022 The Bitcoin Core developers
# Distributed under the MIT software license, see the accompanying
# file COPYING or http://www.opensource.org/licenses/mit-license.php.
#
# Check that all logs are terminated with '\n'
#
# Some logs are continued over multiple lines. They should be explicitly
# commented with /* Continued */

import re
import sys

from subprocess import check_output


def main():
logs_list = check_output(["git", "grep", "--extended-regexp", r"(LogPrintLevel|LogPrintf?)\(", "--", "*.cpp"], universal_newlines=True, encoding="utf8").splitlines()

unterminated_logs = [line for line in logs_list if not re.search(r'(\\n"|/\* Continued \*/)', line)]

if unterminated_logs != []:
print("All calls to LogPrintf(), LogPrint(), LogPrintLevel(), and WalletLogPrintf() should be terminated with \"\\n\".")
print("")

for line in unterminated_logs:
print(line)

sys.exit(1)


if __name__ == "__main__":
main()
29 changes: 0 additions & 29 deletions test/lint/lint-logs.sh

This file was deleted.

0 comments on commit fe16516

Please sign in to comment.