Skip to content

Lara-dsl-deprecation Checkpoint 4 - #20

Closed
lm-sousa wants to merge 1 commit into
lara-dsl-deprecation-checkpoint3from
lara-dsl-deprecation-checkpoint4
Closed

lm-sousa wants to merge 1 commit into
lara-dsl-deprecation-checkpoint3from
lara-dsl-deprecation-checkpoint4

Conversation

@lm-sousa

@lm-sousa lm-sousa commented Sep 3, 2025

Copy link
Copy Markdown
Member

NOT TO BE MERGED. FOR REVIEW PURPOSES ONLY. THIS IS PART OF A STACKED DIFF.

  • Updated comments for clarity and consistency across multiple classes including LineStream, MemoryProfiler, and PatternDetector.
  • Removed commented-out code and unnecessary variables in Table and HeapWindow classes.
  • Enhanced formatting and indentation in ScheduledLinesBuilder and StringLines classes.
  • Simplified constructors and methods in various utility classes to streamline functionality.
  • Improved documentation for methods in XmlElement and XmlNode to better describe parameters and return values.

- Updated comments for clarity and consistency across multiple classes including LineStream, MemoryProfiler, and PatternDetector.
- Removed commented-out code and unnecessary variables in Table and HeapWindow classes.
- Enhanced formatting and indentation in ScheduledLinesBuilder and StringLines classes.
- Simplified constructors and methods in various utility classes to streamline functionality.
- Improved documentation for methods in XmlElement and XmlNode to better describe parameters and return values.
@lm-sousa
lm-sousa requested a review from Copilot September 3, 2025 17:12

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request focuses on code cleanup and documentation improvements across multiple utility classes. The changes update comments for clarity and consistency, remove commented-out code and unnecessary variables, enhance formatting and indentation, simplify constructors and methods, and improve documentation for methods with better parameter and return value descriptions.

Key Changes

  • Removed extensive commented-out code blocks and obsolete variables across many classes
  • Enhanced JavaDoc comments with improved formatting and clearer descriptions
  • Simplified code structure by removing unnecessary complexity in constructors and methods

Reviewed Changes

Copilot reviewed 154 out of 178 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
XmlNode.java Improved comment formatting and removed commented-out exception handling code
XmlElement.java Enhanced JavaDoc comments with better line wrapping for parameter descriptions
Various heap/memory classes Removed commented-out debugging code and simplified constructors
TreeNode-related classes Improved documentation formatting and removed obsolete code
Utility classes General code cleanup with enhanced comment consistency

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread SpecsUtils/src/pt/up/fe/specs/util/treenode/ATreeNode.java
@lm-sousa lm-sousa closed this Jan 18, 2026
@lm-sousa
lm-sousa deleted the lara-dsl-deprecation-checkpoint4 branch January 18, 2026 21:07
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.

3 participants