[PATCH] statfs: don't quote f_type macro names

Zev Weiss zev at bewilderbeest.net
Sun Apr 24 23:52:44 UTC 2016


Hello,

I noticed recently that strace puts quotes around the f_type member of 
struct statfs:

  statfs(".", {f_type="EXT2_SUPER_MAGIC", ...}) = 0

The double-quotes seem to predate git history; ChangeLog-CVS indicates 
they were added intentionally in a commit from Rick Sladkey in 1995 
("Enclose string result in double quotes"), though I don't see any 
reasoning as to why this was done.  Given that strace's output format 
generally seems to aim for an approximate resemblance to C source code, 
it seems like f_type's value would be better off without quotes (since 
it's just a macro, not a string).  Unless there's some more subtle 
reason for the current formatting that I'm not seeing, could the 
attached patch be applied to remove them?


Thanks,
Zev Weiss

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-statfs-don-t-quote-f_type-macro-names.patch
Type: text/x-diff
Size: 1460 bytes
Desc: not available
URL: <http://lists.strace.io/pipermail/strace-devel/attachments/20160424/bfe31c36/attachment.bin>


More information about the Strace-devel mailing list