Merge pull request #37 from EntropyOrg/mohawk2-patch-1

Update StringfiableExtension.pm
EntropyOrg · Nov 4, 2024 · 9f136a5 · 9f136a5
2 parents 1775a63 + 72c6e0c
commit 9f136a5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -63,7 +63,7 @@ jobs:
           target-setup-perl: true
           perl-version: ${{ matrix.perl-version }}
           # Sereal::Encoder can not build in parallel on Windows
-          build-enable-parallel: ${{ ! startsWith(matrix.os, 'windows-') }}
+          build-enable-parallel: ${{ !startsWith(matrix.os,'windows-') && !startsWith(matrix.os,'macos-') }}
       - name: 'ci-dist: target-all after install-dist-perl-deps'
         uses: PDLPorters/devops/github-actions/ci-dist@master
         with:
@@ -72,7 +72,7 @@ jobs:
           target-install-dist-perl-deps: true
           target-test-release-testing: true
           target-test: true
-          build-enable-parallel: ${{ ! startsWith(matrix.os, 'windows-') }}
+          build-enable-parallel: ${{ !startsWith(matrix.os,'windows-') && !startsWith(matrix.os,'macos-') }}
           test-enable-release-testing: ${{ matrix.release-test }}
           test-enable-coverage: ${{ matrix.coverage }}
           github-token: ${{ secrets.GITHUB_TOKEN }}

diff --git a/lib/PDL/StringfiableExtension.pm b/lib/PDL/StringfiableExtension.pm
@@ -18,8 +18,7 @@ use List::AllUtils ();
 			return $_pdl_stringify_temp_single->set(0, $element)->string;
 		}
 		# otherwise
-		my $string = $_pdl_stringify_temp->set(0,0, $element)->string;
-		( $_pdl_stringify_temp->string =~ /\[(.*)\]/ )[0];
+		( $_pdl_stringify_temp->set(0,0, $element)->string =~ /\[(.*)\]/ )[0];
 	}
 }