PDLPorters / pdl

29%

master: 67%

LAST BUILD ON BRANCH nested_tests

branch: nested_tests

CHANGE BRANCH

x

Reset

• nested_tests
• 2.057
• 268-repeat-op-list-context
• 374-pack-D-available
• POD-Dataflow
• TriD_Makefile
• action-notify-irc
• action-notify-irc-build-status
• action-notify-irc-issue-pr
• alienhdf4
• badval-ret-pdl
• benchmarking-matmul
• buildfixes
• c-macros
• centos
• ci-cygwin-only-on-tag
• ci-fix-510
• ci-update
• cirrus-ci
• clang_warnings_2.082
• complex-parse
• complexfuncs
• contrib
• debug-opengl
• dehardcode
• devops-20230507-prima
• double-default
• downstream-testing
• fix-ci
• fix-clang
• fix-empty-vsearch
• fix-rld
• fix_update_doc
• fortran
• freeze_thaw
• from-pdla
• gh-ci
• gha-disttest
• gha-shared-actions
• gha-shared-actions-test-fail
• gha-short-irc-message
• gha-use-expr
• gsl-version-parse
• hainest-assgn_badflag
• index_ops
• inlinepdlpp-package
• io_dumper_dups
• lldfp1
• magnover
• master
• matops-fix
• move-anyval
• ncurses-config
• pack-Q-available
• pdl-srand-seed-func
• pdl_io_fits
• pod-conversion-message
• ppcode-cleanup-01
• pptemplate-doc
• pptemplate-files
• proj6
• r2C-fix
• random-bundle
• rehab_primitive-tests
• remove-c-pdl-new
• reproducibility
• restore-cygwin-in-tag-build-status
• restore-cygwin-in-tag-build-status-alt
• rvals
• scrollbutton
• serialise_dataflow_note
• spelling-errors
• tb
• test-cache
• test-ccache
• test-devops-use-ccache
• test-gha-cache-perl-deps
• test-gha-makeflags-cores
• test-nan-inf-format
• test-tag-for-restore-cygwin-in-tag-build-status
• test-tmp
• undefval
• update-ppport
• upper_sqrt
• use-file-which
• wfits-multiline-history

Build # 5681734546

Build Type

push

github

Committed by djerius

Commit Message

fix incorrect (but working under special case) context for repeat operator

A parenthesized string before the repeat operator normally indicates
a list element repitition, *except* in scalar context, where Perl
will turn it into a string concatenation repetition.  While the
existing code

        t/' . ('*/') x $_ . '*.t'

works because of this special case-ing by Perl, it is confusing to the
reader (and even the author!)

This commit changes this to

        t/' . ('*/' x $_ ). '*.t'

which explicitly uses the string concat repeat context and retains
the parenthesis so the reader doesn't have to worry about operator
precedence.

Run Details

30403 of 105404 relevant lines covered (28.84%)

20227.39 hits per line