<p><b>@pmatilai</b> requested changes on this pull request.</p>

<p>A welcome update, but lets make it really count. Please address the individual comments.</p><hr>

<p>In <a href="https://github.com/rpm-software-management/rpm/pull/827#discussion_r322179100">doc/manual/queryformat</a>:</p>
<pre style='color:#555'>> @@ -136,6 +135,8 @@ readable format in SI resp IEC 80000 standard.
 humansi uses 1K = 1000, 1M = 1000000, ...
 humaniec uses 1K = 1024, 1M = 1048576, ...
 
+The formatting tags are listed in lib/formats.c in structure headerFmt_s.
</pre>
<p>Source code is not user documentation and should not be linked as such. The formats are documented in rpm (8) manual in the QUERY OPTIONS section.</p>

<hr>

<p>In <a href="https://github.com/rpm-software-management/rpm/pull/827#discussion_r322182420">doc/manual/queryformat</a>:</p>
<pre style='color:#555'>> @@ -153,10 +154,10 @@ braces.
 
 \section queryformat_example Example: Viewing the Verify Flags
 
-The following example query is run against dev because I know %verify
-is used there.
+The following example query is run against nss because %verify is used
+there.
</pre>
<p>I agree the old sentence is pretty bad, but this is not much of an improvement. "nss" doesn't go by that name everywhere, the packaging details such as the name and use of %verify are highly distro specific. These days, "systemd" might well be the most ubiquitous name there is, but that doesn't make %verify flags any less distro specific.</p>

<hr>

<p>In <a href="https://github.com/rpm-software-management/rpm/pull/827#discussion_r322183697">doc/manual/queryformat</a>:</p>
<pre style='color:#555'>>  \verbatim
-       rpm -q --qf '[%{filenames} %{fileverifyflags}\n]' dev
+       rpm -q --qf '[%{FILENAMES} %{FILEVERIFYFLAGS:hex}\n]' nss
</pre>
<p>To really improve this example, I'd suggest using something else than FILEVERIFYFLAGS for the data. FILEMODES:octal might be a better example as all packages with files have them, and the values are commonly known, so you could delete the rpmfiles.h reference and values from the document too.</p>

<hr>

<p>In <a href="https://github.com/rpm-software-management/rpm/pull/827#discussion_r322185818">doc/manual/queryformat</a>:</p>
<pre style='color:#555'>> @@ -90,11 +89,11 @@ grep or awk). If you try the obvious,
     rpm -q --queryformat "[%{NAME} %{FILENAMES}\n]" cdp
</pre>
<p>While we're updating package names, here's another candidate. Even I don't remember what "cdp" used to do :D</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/rpm-software-management/rpm/pull/827?email_source=notifications&email_token=ADLPZUZIVFBKXFOSCVJG5ETQIYV5ZA5CNFSM4ISMB77KYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCEBUB6Y#pullrequestreview-285425915">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ADLPZUY7TUBQMRMJA2WKYBTQIYV5ZANCNFSM4ISMB77A">mute the thread</a>.<img src="https://github.com/notifications/beacon/ADLPZU4WRJBFWN2VBDBWJV3QIYV5ZA5CNFSM4ISMB77KYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCEBUB6Y.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/rpm-software-management/rpm/pull/827?email_source=notifications\u0026email_token=ADLPZUZIVFBKXFOSCVJG5ETQIYV5ZA5CNFSM4ISMB77KYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCEBUB6Y#pullrequestreview-285425915",
"url": "https://github.com/rpm-software-management/rpm/pull/827?email_source=notifications\u0026email_token=ADLPZUZIVFBKXFOSCVJG5ETQIYV5ZA5CNFSM4ISMB77KYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCEBUB6Y#pullrequestreview-285425915",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>