Skip to content

C: Refactor analyze.c into src/analyze/#714

Merged
marcoroth merged 4 commits into
mainfrom
cleanup-analyze
Mar 1, 2026
Merged

C: Refactor analyze.c into src/analyze/#714
marcoroth merged 4 commits into
mainfrom
cleanup-analyze

Conversation

@marcoroth

@marcoroth marcoroth commented Oct 22, 2025

Copy link
Copy Markdown
Owner

This pull request reorganizes analyze.c and its related analyze_* files into the src/analyze/ directory.

The big analyze.c file is split into focused modules. Builder infrastructure (the dispatch table, all build_*_node functions, and create_control_node) moves to builders.c.

Control type detection and classification moves to control_type.c. Invalid ERB structure detection becomes invalid_structures.c. Ruby parse error reporting moves to parse_errors.c. What remains in analyze.c is the entry points, ruby analysis, and structure processing logic.

@marcoroth marcoroth changed the title C: Cleanup and refactor analyze.c C: Refactor analyze.c into src/analyze/ Mar 1, 2026
@pkg-pr-new

pkg-pr-new Bot commented Mar 1, 2026

Copy link
Copy Markdown
npx https://pkg.pr.new/@herb-tools/formatter@714
npx https://pkg.pr.new/@herb-tools/language-server@714
npx https://pkg.pr.new/@herb-tools/linter@714

commit: 5b1096f

@github-actions

github-actions Bot commented Mar 1, 2026

Copy link
Copy Markdown

🌿 Interactive Playground and Documentation Preview

A preview deployment has been built for this pull request. Try out the changes live in the interactive playground:


🌱 Grown from commit 5b1096f


✅ Preview deployment has been cleaned up.

@github-actions github-actions Bot added the rust label Mar 1, 2026
@marcoroth marcoroth merged commit 93fe61f into main Mar 1, 2026
32 checks passed
@marcoroth marcoroth deleted the cleanup-analyze branch March 1, 2026 03:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant