Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote: > Quoting the rendered page below: > > > Invoking statx(): > > This is an odd header for a man page. Usually this would start as > > DESCRIPTION > > (in bold). Yes, I know. But it seems to make more sense this way round. Putting the struct definition before the description of how to access it means that I can refer to it in the definitions of the flags that you OR into the mask and means you get to the struct much faster. Maybe I should put it under the SYNOPSIS section? > > [*] By absolute path. > > Odd way to enumerate in a man page - I'll leave it to the linux-man regulars > to point out the right way, because I don't know it off-hand. Me neither. > > AT_EMPTY_PATH (since Linux 2.6.39) > > All these have been there since statx was added, so we can remove the > annotation. You're not the first to note this. I've already taken that out. > > STATX_BASIC_STATS [The stuff in the normal stat struct] > > Please enumerate the actual fields. They're enumerated immediately prior. I've changed it to: STATX_BASIC_STATS [All of the above] > > STATX_ALL [All currently available stuff] > > How about "All currently available fields" Sure. David -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html