search

nasa / bplib

Apache License 2.0
27 stars 13 forks source link

Fix #217, add handler for bplib_os_get_dtntime_ms #218

Closed jphickey closed 1 year ago

jphickey commented 1 year ago

Describe the contribution Implements a simple handler to translate the 32-bit return value into 64 bits. Of course this reduces the possible range, but should be good enough for the vast majority of coverage testing.

Fixes #217

Testing performed Build and run all tests (including new/unmerged mpool test cases)

Expected behavior changes The stub for bplib_os_get_dtntime_ms() does not throw an error when used in default configuration.

System(s) tested on Ubuntu

Additional context This only matters for test cases that need to call this stub function.

Contributor Info - All information REQUIRED for consideration of pull request Joseph Hickey, Vantage Systems, Inc.

jphickey commented 1 year ago

Reviewed and provisionally approved at 2022-11-30 CCB, needs independent review before merge.

dzbaker commented 1 year ago

PR has been reviewed, ready to move from Provisional Approval to full Approval.