Skip to content

fix: B::CV->START line for Fennec::Lite (FENNEC_ITEM)#733

Merged
fglock merged 1 commit into
masterfrom
fix/b-cv-start-line
May 15, 2026
Merged

fix: B::CV->START line for Fennec::Lite (FENNEC_ITEM)#733
fglock merged 1 commit into
masterfrom
fix/b-cv-start-line

Conversation

@fglock
Copy link
Copy Markdown
Owner

@fglock fglock commented May 15, 2026

Summary

  • Implement real B::CV->START source locations so B::svref_2object($cv)->START->line matches Perl for filtering (e.g. Fennec::Lite FENNEC_ITEM).
  • Previously the bundled B stub always returned line 0, so every test group matched the numeric filter and t/RunByLine.t failed with got: 'correcu' (string increment after "correct").

Test plan

  • make
  • timeout 600 ./jcpan -t Fennec::Lite

Fennec::Lite filters tests with start_line from B::svref_2object($cv)->START->line.
The bundled B stub always returned line 0, so every group matched and RunByLine.t
failed (correcu from "correct"++).

- Track cvStartFile/cvStartLine on RuntimeCode; set for InterpretedCode, CompiledCode,
  parse-time placeholders, and JVM anon subs via makeCodeObject.
- Add Internals::jperl_cv_start_location; wire B::CV::START to use it.

Fixes jcpan -t Fennec::Lite.

Generated with [Cursor](https://cursor.com/docs)

Co-Authored-By: Cursor <noreply@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@fglock fglock merged commit b918168 into master May 15, 2026
3 of 4 checks passed
@fglock fglock deleted the fix/b-cv-start-line branch May 15, 2026 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant