Currently we load ignore files that are "$dir/$name" --- always with /. Make that cross-platform.

Currently, all the tests of traversal use the disk. Add tests of a non-disk FS.

• Permit the caller to specify extra ignores that will be applied in addition to those from an ignore file in a particular directory
• Make sure those are force-ignored, even if an ignore file unignores one.

It would be nice for users to not get that whole header in their namespaces.

Follows #2.

In t/globstari-matcher-t.cpp, turn off -fvar-tracking-assignments if the compiler supports doing so. This will make compilation of that file faster.

A program you can run from the command line to output a single log line. That way scripts will be able to log in the same format as C programs.

Permit defining a log domain that will not be affected by V=... or FOO_DEBUG='*:10', but that can be enabled with FOO_DEBUG='levelname:10'.

Permit bypassing the ignore mechanism for specific entries in a directory.

• throw?
• Default to "*"?

As it stands, if you omit the needle (e.g., pass {}), it looks like you silently get no results (no calls to processEntry()).

a la Alexandrescu

Maybe using https://github.com/ricab/scope_guard ?

The assertions in TEST_CASE mean that a test will succeed even if the user didn't specify any tests.

I think something like this:

struct Printer {
    operator=(LogStringFormatter&) { ... }
} g_PrintNow;

#define LOG_S(...) \
    g_PrintNow = LogStringFormatter(__VA_ARGS__)
...
LOG_S(INFO) << "Foo " << i;

That will make it easier to, e.g., use adapters.

When a file is ignored, call a (default nop) virtual function. That way subclasses can see the ignored files as well as the non-ignored files.

They currently do not.

diff --git a/3rdparty/smallcxx/logging.hpp b/3rdparty/smallcxx/logging.hpp
index 95202f8..fc03ba9 100644
--- a/3rdparty/smallcxx/logging.hpp
+++ b/3rdparty/smallcxx/logging.hpp
@@ -93,7 +93,7 @@ void vlogMessage(const std::string& domain,
 /// Usage example: `LOG_F(INFO, "foo %s", some_string);`
 /// @note `LOG_F(SILENT, ...)` is forbidden.
 #define LOG_F(level, format, ...) \
-    LOG_F_DOMAIN(SMALLCXX_LOG_DOMAIN_NAME, level, (format), ## __VA_ARGS__);
+    LOG_F_DOMAIN(SMALLCXX_LOG_DOMAIN_NAME, level, format, ## __VA_ARGS__);
 
 /// Log, with a particular log domain.
 #define LOG_F_DOMAIN(domain, level, format, ...) \
@@ -107,11 +107,11 @@ void vlogMessage(const std::string& domain,
         /* print it */ \
         if(LOG_##level == LOG_PRINT) { \
             if(getLogLevel(domain) != LOG_SILENT) { \
-                printf((format), ## __VA_ARGS__); \
+                printf(format "\n", ## __VA_ARGS__); \
             } \
         } else if(LOG_##level == LOG_PRINTERR) { \
             if(getLogLevel(domain) != LOG_SILENT) { \
-                fprintf(stderr, (format), ## __VA_ARGS__); \
+                fprintf(stderr, format "\n", ## __VA_ARGS__); \
             } \
         } else { \
             logMessage(domain, LOG_##level, __FILE__, __LINE__, __func__, \

ag and editorconfig aren't quite what I'm looking for.

• Ability to traverse virtual (no readdir()) trees
• Ability to turn ignores on and off

            LOG_F_DOMAIN(" test", LOG, "=> Starting test %s", #fn); \

doesn't appear, even with DEBUG='*:7'

Used by setVerbosityFromEnvironment when given an env var name.

Call smallcxxlog multiple times from a script and you may see, e.g.:

[780437.384507222] 288405   ERROR ./630-cmdline-checko:73   test_subdir          Command failed (code 1): [ -e /tmp/tmp.qtbeIhe7s4/dir/first ]
[780437.411007634] 288419   ERROR            common.sh:227  readable_file_ok     Command failed (code 1): [ -f /tmp/tmp.qtbeIhe7s4/dir/first ]
[780437.434050256] 288433   ERROR            common.sh:228  readable_file_ok     Command failed (code 1): [ -r /tmp/tmp.qtbeIhe7s4/dir/first ]
                   ^^^^^^  different PIDs each time

even though the script's PID hasn't changed. Permit smallcxxlog to take the PID as a parameter.

Instead of userdata, let callers pass pointers to Entry subclasses.