- changed status to open
- removed comment
avoid allocting temp memory twice in EOS Omni
instead of storing a copy of the data in the Fortran module store Fortran pointers.
Keyword:
Comments (7)
-
reporter -
- removed comment
Patch looks ok, but I a) didn't test it and b) don't know how to work these Fortran pointers (otherwise I wouldn't have written the now removed comments :). If this passes the tests (and is indeed tested), please commit. Leaving as 'review' since you might want to find someone with more Fortran knowledge to review this.
-
reporter - removed comment
It passes all existing EOSOmni table reader tests -- of which there are none :-). Seriously though I added a tiny function to dump the read in data into an ascii file and the ascii file compares identical before and after the patch. This is not urgent though and we can use to hunt around for a EOS Omni table reader test suite.
-
- removed comment
In this case ok to commit - but I think this is not important enough to go into the next release, so please hold back until after it.
-
- changed milestone to ET_2012_11
- removed comment
-
reporter - changed status to resolved
- removed comment
applied as rev 62 of EOS_Omni.
-
reporter - edited description
- changed status to closed
- Log in to comment