Skip to content

Add DicomTime::millisecond#665

Merged
Enet4 merged 3 commits intomasterfrom
imp/core/time-millis
Jul 25, 2025
Merged

Add DicomTime::millisecond#665
Enet4 merged 3 commits intomasterfrom
imp/core/time-millis

Conversation

@Enet4
Copy link
Copy Markdown
Owner

@Enet4 Enet4 commented Jul 23, 2025

This is a non-breaking addition to dicom_core to mitigate #663 until a better resolution is made.

Summary

  • [core] change DicomTime::fraction_and_precision to return non-references
  • [core] add DicomTime::millisecond, which returns the number of milliseconds of the fraction of a second, if the fraction is precise to the millisecond or more.

…ences

not a breaking change because the method is pub(crate) visible
@Enet4 Enet4 added enhancement A-lib Area: library C-core Crate: dicom-core labels Jul 23, 2025
returns the fraction of a second in millisecond if the time is precise enough
@Enet4 Enet4 force-pushed the imp/core/time-millis branch from fcf012d to 68ea02c Compare July 23, 2025 17:33
@Enet4 Enet4 merged commit fa676bc into master Jul 25, 2025
5 checks passed
@Enet4 Enet4 deleted the imp/core/time-millis branch December 17, 2025 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-lib Area: library C-core Crate: dicom-core enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant