On 2023/07/10 15:28, lijiang wrote:
> @@ -112,8 +112,8 @@ vmware_vmss_init(char *filename, FILE *ofp)
>          }
>
>          if (fread(grps, sizeof(cptgroupdesc), grpsize, fp) != grpsize) {
> -               error(INFO, LOGPRX"Failed to read '%s': [Error %d]
%s\n",
> -                     filename, errno, strerror(errno));
> +               error(INFO, LOGPRX"Failed to read '%s' from file
'%s':
> [Error %d] %s\n",
> +                     "cptgroupdesc", filename, errno, strerror(errno));
>
                              ^^^^^^^^^^^^
 
 Although the above code is good, I tend to output constant strings directly
 rather than using the '%s' to print.
 
 Anyway, the other changes still look fine to me. So: Ack.
 
Well, maybe it's grep-friendly, maybe it's a bit strange to expand the 
existing 'name' variable for code consistency.  Let's take the patch as 
it is in this case.
https://github.com/crash-utility/crash/commit/b76e116c50ffc228ebc08eb8de3...
Thanks,
Kazu