Skip to content

Commit 63f87c8

Browse files
committed
cleanup
1 parent 5d14a1a commit 63f87c8

File tree

6 files changed

+10
-14
lines changed

6 files changed

+10
-14
lines changed

dev/prompts/debug-re-pat-test.md

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -55,25 +55,25 @@
5555
### Basic Run Command
5656
```bash
5757
# Run with all necessary flags
58-
PERL_SKIP_BIG_MEM_TESTS=1 JPERL_UNIMPLEMENTED=warn JPERL_LARGECODE=refactor ./jperl t/re/pat.t
58+
PERL_SKIP_BIG_MEM_TESTS=1 JPERL_UNIMPLEMENTED=warn ./jperl t/re/pat.t
5959
```
6060

6161
### Check Where Test Stops
6262
```bash
6363
# See last 20 test results
64-
PERL_SKIP_BIG_MEM_TESTS=1 JPERL_UNIMPLEMENTED=warn JPERL_LARGECODE=refactor ./jperl t/re/pat.t 2>&1 | grep -E "(^ok |^not ok |planned.*ran)" | tail -20
64+
PERL_SKIP_BIG_MEM_TESTS=1 JPERL_UNIMPLEMENTED=warn ./jperl t/re/pat.t 2>&1 | grep -E "(^ok |^not ok |planned.*ran)" | tail -20
6565
```
6666

6767
### See Error Details
6868
```bash
6969
# Show last 30 lines including errors
70-
PERL_SKIP_BIG_MEM_TESTS=1 JPERL_UNIMPLEMENTED=warn JPERL_LARGECODE=refactor ./jperl t/re/pat.t 2>&1 | tail -30
70+
PERL_SKIP_BIG_MEM_TESTS=1 JPERL_UNIMPLEMENTED=warn ./jperl t/re/pat.t 2>&1 | tail -30
7171
```
7272

7373
### Debug Specific Test
7474
```bash
7575
# Extract specific test number (replace 294 with test number)
76-
PERL_SKIP_BIG_MEM_TESTS=1 JPERL_UNIMPLEMENTED=warn JPERL_LARGECODE=refactor ./jperl t/re/pat.t 2>&1 | grep -A 10 "^not ok 294"
76+
PERL_SKIP_BIG_MEM_TESTS=1 JPERL_UNIMPLEMENTED=warn ./jperl t/re/pat.t 2>&1 | grep -A 10 "^not ok 294"
7777
```
7878

7979
## Known Issues and Patterns
@@ -112,7 +112,6 @@ PERL_SKIP_BIG_MEM_TESTS=1 JPERL_UNIMPLEMENTED=warn JPERL_LARGECODE=refactor ./jp
112112

113113
- `PERL_SKIP_BIG_MEM_TESTS=1` - Skip memory-intensive tests (Long Monsters)
114114
- `JPERL_UNIMPLEMENTED=warn` - Warn on unimplemented features instead of dying
115-
- `JPERL_LARGECODE=refactor` - Handle large methods by refactoring code blocks
116115

117116
## Quick Wins to Continue
118117

dev/prompts/fix-pack-test-failures.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
**Total tests:** 14,724
88

99
**Test Environment:**
10-
- Requires: `JPERL_UNIMPLEMENTED=warn JPERL_LARGECODE=refactor`
10+
- Requires: `JPERL_UNIMPLEMENTED=warn`
1111
- All 14,724 tests run to completion (no crashes)
1212

1313
---

dev/prompts/high-yield-test-analysis-strategy.md

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -347,7 +347,7 @@ diff <(perl -e 'TEST_CODE') <(./jperl -e 'TEST_CODE')
347347
1. **Identify the exact failing test:**
348348
```bash
349349
# Find which specific test is failing
350-
JPERL_LARGECODE=refactor ./jperl t/op/pack.t 2>&1 | grep -A 3 "^not ok NUMBER"
350+
./jperl t/op/pack.t 2>&1 | grep -A 3 "^not ok NUMBER"
351351
```
352352

353353
2. **Find the test code in the source:**
@@ -399,7 +399,7 @@ diff <(perl -e 'TEST_CODE') <(./jperl -e 'TEST_CODE')
399399
**Red Flags:**
400400
- If extraction works but full suite fails → test interaction or state issue
401401
- If extraction fails differently → you extracted the wrong test
402-
- If extraction passes → file size/compilation issue (e.g., JPERL_LARGECODE)
402+
- If extraction passes → file size/compilation issue
403403

404404
**Template for extraction:**
405405
```perl
@@ -428,7 +428,7 @@ print "# Tests completed\n";
428428
**Remember:**
429429
- PerlOnJava doesn't cache code between runs (no stale bytecode)
430430
- Always rebuild after code changes: `./gradlew clean shadowJar`
431-
- Use `JPERL_LARGECODE=refactor` if original test needed it
431+
- Use `git add -a files` one by one
432432

433433
### Technique 5: Test Count Analysis
434434
**Pattern:** Track test improvements to verify fixes
@@ -1165,7 +1165,6 @@ jq '.results["TESTFILE.t"]' out.json
11651165
./jperl t/op/TESTFILE.t 2>&1 | grep -A 3 "^not ok NUMBER"
11661166

11671167
# Run with special environment variables
1168-
JPERL_LARGECODE=refactor ./jperl t/op/pack.t # Handle large methods by refactoring blocks
11691168
JPERL_UNIMPLEMENTED=warn ./jperl t/op/TESTFILE.t # Warn on unimplemented features instead of dying
11701169
```
11711170

dev/tools/analyze_pack_failures.pl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77

88
# Run pack.t and capture output
99
print "Running pack.t tests...\n";
10-
my $output = `JPERL_UNIMPLEMENTED=warn JPERL_LARGECODE=refactor ./jperl t/op/pack.t 2>&1`;
10+
my $output = `JPERL_UNIMPLEMENTED=warn ./jperl t/op/pack.t 2>&1`;
1111

1212
# Parse test results
1313
my @lines = split /\n/, $output;

dev/tools/perl_test_runner.pl

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -251,8 +251,6 @@ sub run_single_test {
251251
re/pat.t
252252
| op/repeat.t }x
253253
? "-Xss256m" : "";
254-
local $ENV{JPERL_LARGECODE} = $test_file =~ m{ re/pat.t | re/pat_advanced.t | op/signatures.t | t/op/pack.t }x
255-
? "refactor" : "";
256254

257255
# Skip memory-intensive tests (e.g., Long Monsters in re/pat.t with 300KB strings)
258256
# These tests can crash the JVM with StackOverflowError during regex backtracking

docs/FEATURE_MATRIX.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -659,7 +659,7 @@ This section is being worked on.
659659
## Optimizations
660660

661661
-**Cached string/numeric conversions**: Numification caching is implemented.
662-
- 🚧 **Java segment size limitation**: Java bytecode segments are limited to 64k bytes.
662+
- 🚧 **Java segment size limitation**: A workaround is implemented to Java 64k bytes segment limit.
663663
-**Inline "constant" subroutines optimization**: Optimization for inline constants is not yet implemented.
664664
-**Overload optimization**: Preprocessing in overload should be cached.
665665
-**I/O optimization**: Use low-level readline to optimize input.

0 commit comments

Comments
 (0)