Skip to content

Fix Mem-leak and CMake issues#140

Merged
sanathkeshav merged 5 commits intoDataAnalyticsEngineering:developfrom
Snapex2409:small-fixes
May 7, 2026
Merged

Fix Mem-leak and CMake issues#140
sanathkeshav merged 5 commits intoDataAnalyticsEngineering:developfrom
Snapex2409:small-fixes

Conversation

@Snapex2409
Copy link
Copy Markdown
Contributor

@Snapex2409 Snapex2409 commented May 6, 2026

  • Fixes a mem-leak encountered during Reader::ReadMS
  • Updates CMake files to support a wider range of systems. Depending on the system, the previous version would sometimes fail to compile.

Checklist:

  • I made sure that the CI passed before I ask for a review.
  • I added a summary of the changes (compared to the last release) in the CHANGELOG.md.
  • If necessary, I made changes to the documentation and/or added new content.
  • I will remember to squash-and-merge, providing a useful summary of the changes of this PR.

@Snapex2409 Snapex2409 marked this pull request as ready for review May 6, 2026 17:48
@sanathkeshav sanathkeshav merged commit e6d89fc into DataAnalyticsEngineering:develop May 7, 2026
8 checks passed
@Snapex2409 Snapex2409 deleted the small-fixes branch May 7, 2026 08:24
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.

2 participants