Readability of warning message

albert-github · albert-github · commit e1f55871fa67 · 2020-09-04T13:33:43.000+02:00
A warning like:
```
warning: source examples is not a readable file or directory... skipping.
```
is hard to understand, would be better to have:
```
warning: source 'examples' is not a readable file or directory... skipping.
```
diff --git a/src/doxygen.cpp b/src/doxygen.cpp
@@ -9473,7 +9473,7 @@ static int readDir(QFileInfo *fi,
         {
           if (errorIfNotExist)
           {
-            warn_uncond("source %s is not a readable file or directory... skipping.\n",cfi->absFilePath().data());
+            warn_uncond("source '%s' is not a readable file or directory... skipping.\n",cfi->absFilePath().data());
           }
         }
         else if (cfi->isFile() &&
@@ -9553,7 +9553,7 @@ int readFileOrDirectory(const char *s,
       {
         if (errorIfNotExist)
         {
-          warn_uncond("source %s is not a readable file or directory... skipping.\n",s);
+          warn_uncond("source '%s' is not a readable file or directory... skipping.\n",s);
         }
       }
       else if (!Config_getBool(EXCLUDE_SYMLINKS) || !fi.isSymLink())

Original file line number	Diff line number	Diff line change
`@@ -9473,7 +9473,7 @@ static int readDir(QFileInfo *fi,`
`9473`	`9473`	`{`
`9474`	`9474`	`if (errorIfNotExist)`
`9475`	`9475`	`{`
`9476`		`- warn_uncond("source %s is not a readable file or directory... skipping.\n",cfi->absFilePath().data());`
	`9476`	`+ warn_uncond("source '%s' is not a readable file or directory... skipping.\n",cfi->absFilePath().data());`
`9477`	`9477`	`}`
`9478`	`9478`	`}`
`9479`	`9479`	`else if (cfi->isFile() &&`
`@@ -9553,7 +9553,7 @@ int readFileOrDirectory(const char *s,`
`9553`	`9553`	`{`
`9554`	`9554`	`if (errorIfNotExist)`
`9555`	`9555`	`{`
`9556`		`- warn_uncond("source %s is not a readable file or directory... skipping.\n",s);`
	`9556`	`+ warn_uncond("source '%s' is not a readable file or directory... skipping.\n",s);`
`9557`	`9557`	`}`
`9558`	`9558`	`}`
`9559`	`9559`	`else if (!Config_getBool(EXCLUDE_SYMLINKS) \|\| !fi.isSymLink())`