Bram

Results 45 comments of Bram

> Why not to respect and use POD where ever possible when GitHub renders POD too? 🙂 > > GitHub formats POD README's as good as Markdown, so I see...

Is it possible to add test case for this?

If a test case is not possible, is an example - that could be included in the commit message - possible? That is: can you add a - non portable...

I'm a bit confused by the commit message (but that is probably just be me since I lack prior knowledge)... What I don't get: ``` [&foo] [$bar] # defining scope...

Hmm.. I don't think I'm fully in favor for this.. I think SECURITY.md should stay in the root and should be included in the release. It's a lot easier to...

Reducing t/50_inplace.t of File::Replace to a minimal test case: ``` #!/usr/bin/perl -l use strict; use warnings; use POSIX qw/dup dup2/; sub s1 { print "\teof() is " . (eof() ?...

> What was `$Config{gccversion}` for your failing perl ? Using the toolchain that came with latest Strawberry Perl, so gcc 8.3.0 (so according to your follow up reply that should...

Just a small heads up: I'm planning on doing some testing with this and some reviewing; (so if possible: wait with merging until I had a chance to test/review)

I think the first commits can be merged as-is, For the whitespace cleanup commit, skimming the files: - op.h doesn't look good: - lines 11-40 are misaligned - lines 111-167:...

Looing at a cpan test report[^1]: ``` # Failed test 'get magic with op_info == 1 gets the right op info' # at t/18-opinfo.t line 79. # got: 'padsv_store' #...