Skip to content

Addition of getstored - #22

Merged
jvdp1 merged 3 commits into
masterfrom
getstored
Feb 16, 2026
Merged

Addition of getstored#22
jvdp1 merged 3 commits into
masterfrom
getstored

Conversation

@jvdp1

@jvdp1 jvdp1 commented Feb 15, 2026

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 adds a getstored method to the table classes that returns all stored values up to the filled count. The PR also refactors internal methods to use direct array access instead of the bounds-checking get method for performance, adds explicit subroutine names to end statements, and updates Makefile compiler flags to support C preprocessor directives.

Changes:

  • Added getstored methods to both scalar and array table types that return the filled portion of the internal storage array
  • Refactored writetable, writestream, and writehdf5 methods to use direct stored array access instead of get method calls for better performance
  • Added explicit subroutine names to all end subroutine statements in test file for clarity
  • Updated Makefile compiler flags to include -fpp/-cpp for C preprocessor support and changed standard from f08 to f18

Reviewed changes

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

File Description
src/modtable.fypp Added getstored method declarations and implementations for both scalar and array table types; refactored write methods to use direct array access
test/test_table_stream.fypp Added test cases for the new getstored method in all test subroutines; added explicit names to end subroutine statements
Makefile Updated compiler flags: added preprocessor flags (-fpp/-cpp) and changed Fortran standard from f08 to f18 for ifort

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread test/test_table_stream.fypp Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@jvdp1
jvdp1 merged commit 7bbcf7d into master Feb 16, 2026
16 checks passed
@jvdp1
jvdp1 deleted the getstored branch February 16, 2026 07:27
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