Problems Raised in rjbs's review of p.r.2 that I will defer to later, but already planned, p.r.s $Podroot - extraneous definition? (https://github.com/Perl/perl5/pull/18950#discussion_r663545538) * $Podroot will become $self->{Podroot} in "5 of 5" (https://github.com/Perl/perl5/pull/19050) Documentation of all subroutines in Pod::Html::Util * Handled mainly in bc4a9d813672ab84362aa3371710f75c49a6f905 with touch-ups in subsequent commits merged into blead in ca5ef5cf0c031c02848b858299784105a60e33e8 (Aug 14) Pod-Html Nits the Picking of Which Will Be Deferred until after Pull Request 5 (P-HNtPoWWBDuaPR5) ##### Remaining unresolved conversations with rjbs in https://github.com/Perl/perl5/pull/19050 chdir || die versus open or die Wordspace of not between 'if' or 'while' and '(' Cuddled else's warnings on Verbose (https://github.com/Perl/perl5/pull/18950#discussion_r663545765) ##### Concerns about @ARGV handling See Oodler's comments in: https://github.com/jkeenan/perl5/pull/4#discussion_r632271870 https://github.com/jkeenan/perl5/pull/7#issuecomment-841001030 and rjbs comments in: https://github.com/Perl/perl5/pull/18950#discussion_r663545308 ##### installhtml: uninit value warnings reported by sisyphus in: https://github.com/Perl/perl5/pull/19000#issuecomment-886184762 These should be cleared up by "5 of 5" pull request: VERIFY. * Documented as https://github.com/Perl/perl5/issues/19052 lib/Pod/Html.pm: Simplify trailing 'if' per rjbs comment in https://github.com/Perl/perl5/pull/18977#discussion_r671734364 * Handled in c0ac28175b4f88934f5b20131fbcd727c858af8d (committed Aug 14), but double check after https://github.com/Perl/perl5/pull/19050 is merged. ##### Added remaining-issues-20210820.txt https://github.com/Perl/perl5/pull/19050#discussion_r693202078 lib/Pod/Html.pm: use postfix dereference generate_cache(): updirs_only; rjbs suggestion about spelling https://github.com/Perl/perl5/pull/19050#discussion_r693198901