Add flags to make listing easy to diff

follow-up to the discussion originally in https://github.com/OCamlPro/gnucobol/pull/75, implementing part of https://sourceforge.net/p/gnucobol/feature-requests/294/

New flags:

-fno-ttimestamp: suppress time stamp
-fttitle=: display <title> instead of package name and version</li> </ul> </div> </div> <div class="comment"> <div class="user"> <a rel="noreferrer nofollow" target="_blank" href="https://github.com/lefessan"><img src="https://avatars.githubusercontent.com/u/514957?v=4" />lefessan</a> commented <strong> 1 year ago</strong> </div> <div class="markdown-body"> <p>I added <code>-ftversion</code> to avoid escaping problems with the space in the title in <code>atlocal.in</code>...</p> </div> </div> <div class="comment"> <div class="user"> <a rel="noreferrer nofollow" target="_blank" href="https://github.com/lefessan"><img src="https://avatars.githubusercontent.com/u/514957?v=4" />lefessan</a> commented <strong> 1 year ago</strong> </div> <div class="markdown-body"> <p>I still have many tests in <code>listing.at</code> to modify...</p> </div> </div> <div class="comment"> <div class="user"> <a rel="noreferrer nofollow" target="_blank" href="https://github.com/GitMensch"><img src="https://avatars.githubusercontent.com/u/6699539?v=4" />GitMensch</a> commented <strong> 1 year ago</strong> </div> <div class="markdown-body"> <p>Just stumbled over that: one of the listing tests has 2x identical prog.cob with <code>PROCEDURE DIVISION USING PAR-IN RETURNING PAR-OUT.</code> - if possible please drop the second AT_DATA when adjusting listing.at</p> </div> </div> <div class="comment"> <div class="user"> <a rel="noreferrer nofollow" target="_blank" href="https://github.com/codecov-commenter"><img src="https://avatars.githubusercontent.com/u/65553080?v=4" />codecov-commenter</a> commented <strong> 1 year ago</strong> </div> <div class="markdown-body"> <h2><a href="https://app.codecov.io/gh/OCamlPro/gnucobol/pull/105?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=OCamlPro">Codecov</a> Report</h2> <blockquote> <p>Merging <a href="https://app.codecov.io/gh/OCamlPro/gnucobol/pull/105?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=OCamlPro">#105</a> (afeb5ca) into <a href="https://app.codecov.io/gh/OCamlPro/gnucobol/commit/4b8452abf1cafb904f1fe6966cbeeaa2a5f9d10a?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=OCamlPro">gcos4gnucobol-3.x</a> (4b8452a) will <strong>increase</strong> coverage by <code>0.01%</code>. The diff coverage is <code>92.85%</code>.</p> </blockquote> <p>:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please <a href="https://github.com/apps/codecov">install the Github App Integration</a> for your organization. <a href="https://about.codecov.io/blog/codecov-is-updating-its-github-integration/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=OCamlPro">Read more</a>.</p> <pre><code class="language-diff">@@ Coverage Diff @@ ## gcos4gnucobol-3.x #105 +/- ## ===================================================== + Coverage 65.17% 65.19% +0.01% ===================================================== Files 31 31 Lines 58384 58393 +9 Branches 15380 15384 +4 ===================================================== + Hits 38051 38067 +16 + Misses 14402 14396 -6 + Partials 5931 5930 -1 </code></pre> <table> <thead> <tr> <th><a href="https://app.codecov.io/gh/OCamlPro/gnucobol/pull/105?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=OCamlPro">Impacted Files</a></th> <th>Coverage Δ</th> <th></th> </tr> </thead> <tbody> <tr> <td><a href="https://app.codecov.io/gh/OCamlPro/gnucobol/pull/105?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=OCamlPro#diff-Y29iYy9jb2JjLmM=">cobc/cobc.c</a></td> <td><code>71.53% <88.88%> (+0.25%)</code></td> <td>:arrow_up:</td> </tr> <tr> <td><a href="https://app.codecov.io/gh/OCamlPro/gnucobol/pull/105?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=OCamlPro#diff-Y29iYy9mbGFnLmRlZg==">cobc/flag.def</a></td> <td><code>100.00% <100.00%> (ø)</code></td> <td></td> </tr> </tbody> </table> <p>:mega: We’re building smart automated test selection to slash your CI/CD build times. <a href="https://about.codecov.io/iterative-testing/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=OCamlPro">Learn more</a></p> </div> </div> <div class="comment"> <div class="user"> <a rel="noreferrer nofollow" target="_blank" href="https://github.com/lefessan"><img src="https://avatars.githubusercontent.com/u/514957?v=4" />lefessan</a> commented <strong> 1 year ago</strong> </div> <div class="markdown-body"> <blockquote> <p>Just stumbled over that: one of the listing tests has 2x identical prog.cob with <code>PROCEDURE DIVISION USING PAR-IN RETURNING PAR-OUT.</code> - if possible please drop the second AT_DATA when adjusting listing.at</p> </blockquote> <p>I think I found it and removed it.</p> </div> </div> <div class="comment"> <div class="user"> <a rel="noreferrer nofollow" target="_blank" href="https://github.com/lefessan"><img src="https://avatars.githubusercontent.com/u/514957?v=4" />lefessan</a> commented <strong> 1 year ago</strong> </div> <div class="markdown-body"> <p>Argh, tests run fine with <code>autofonce</code> but not with <code>autotest</code>, I need to investigate the different behavior...</p> </div> </div> <div class="comment"> <div class="user"> <a rel="noreferrer nofollow" target="_blank" href="https://github.com/GitMensch"><img src="https://avatars.githubusercontent.com/u/6699539?v=4" />GitMensch</a> commented <strong> 1 year ago</strong> </div> <div class="markdown-body"> <blockquote> <p>Argh, tests run fine with <code>autofonce</code> but not with <code>autotest</code>, I need to investigate the different behavior...</p> </blockquote> <p>Just inspected the testsuite.log: it is exactly as I've thought:</p> <pre><code class="language-diff">-GnuCOBOL V.R.P prog.cob +GnuCOBOL V.R.P prog.cob </code></pre> <p>To fix that change the place where <code>cb_listing_with_timestamp</code> is checked (should be in <code>print_program_header()</code>) and the tests will pass. Note that this directly allows to give either the title or the filename more space, too.</p> </div> </div> <div class="comment"> <div class="user"> <a rel="noreferrer nofollow" target="_blank" href="https://github.com/lefessan"><img src="https://avatars.githubusercontent.com/u/514957?v=4" />lefessan</a> commented <strong> 1 year ago</strong> </div> <div class="markdown-body"> <p>I will commit this version to SVN as soon as the CI passes, to be able to switch to the other PR. There are still tests with spaces at the end of headers, that I fixed using a quadrigraph (<code>@&t@</code>) to prevent autoconf from removing them. I preferred not to change the printing because the spaces are generated by <code>%-26.26s</code> in <code>printf</code> and I was not sure of the intention.</p> </div> </div> <div class="comment"> <div class="user"> <a rel="noreferrer nofollow" target="_blank" href="https://github.com/lefessan"><img src="https://avatars.githubusercontent.com/u/514957?v=4" />lefessan</a> commented <strong> 1 year ago</strong> </div> <div class="markdown-body"> <p>( I just pushed a commit in <code>autofonce</code> to correctly handle spaces, in case you use it)</p> </div> </div> <div class="comment"> <div class="user"> <a rel="noreferrer nofollow" target="_blank" href="https://github.com/GitMensch"><img src="https://avatars.githubusercontent.com/u/6699539?v=4" />GitMensch</a> commented <strong> 1 year ago</strong> </div> <div class="markdown-body"> <blockquote> <p>I preferred not to change the printing because the spaces are generated by <code>%-26.26s</code> in <code>printf</code> and I was not sure of the intention.</p> </blockquote> <p>The point was that there is a trailing timestamp, right adjusted. If <code>-fno-ttimestamp</code> is used we currently printf the low-value; but when this is changed to not adjustt the timestamp var but instead adjust the printing and right-justify the sourcename, then we should not have any trailing spaces left.</p> </div> </div> <div class="comment"> <div class="user"> <a rel="noreferrer nofollow" target="_blank" href="https://github.com/GitMensch"><img src="https://avatars.githubusercontent.com/u/6699539?v=4" />GitMensch</a> commented <strong> 1 year ago</strong> </div> <div class="markdown-body"> <p>LGTM (we can right-adjust the filename for <code>-fno-ttimestamp</code> later)</p> </div> </div> <div class="comment"> <div class="user"> <a rel="noreferrer nofollow" target="_blank" href="https://github.com/lefessan"><img src="https://avatars.githubusercontent.com/u/514957?v=4" />lefessan</a> commented <strong> 1 year ago</strong> </div> <div class="markdown-body"> <p>Merged with whitespaces removed at the end of line</p> </div> </div> <div class="page-bar-simple"> </div> <div class="footer"> <ul class="body"> <li>© <script> document.write(new Date().getFullYear()) </script> Githubissues.</li> <li>Githubissues is a development platform for aggregating issues.</li> </ul> </div> <script src="https://cdn.jsdelivr.net/npm/jquery@3.5.1/dist/jquery.min.js"></script> <script src="/githubissues/assets/js.js"></script> <script src="/githubissues/assets/markdown.js"></script> <script src="https://cdn.jsdelivr.net/gh/highlightjs/cdn-release@11.4.0/build/highlight.min.js"></script> <script src="https://cdn.jsdelivr.net/gh/highlightjs/cdn-release@11.4.0/build/languages/go.min.js"></script> <script> hljs.highlightAll(); </script> </body> </html>

OCamlPro / gnucobol

Add flags to make listing easy to diff #105