Project

General

Profile

Actions
Copy link

Patch #895

closed

OFDateTime/OFTime: Support for optional fractional part of a second in setISOFormattedXXX()

Added by Jörg Riesmeier over 6 years ago. Updated 5 days ago.

Status:
Closed
Priority:
Normal
Assignee:
Harald Roesen
Category:
Library
Target version:
-
Start date:
2019-08-07
Due date:
% Done:

0%

Estimated time:
Module:
ofstd
Operating System:
Compiler:

Description

GitHub user "PavelTrushkin" says:

i have reworked OFTime::setISOFormattedTime and modified OFDateTime::setISOFormattedDateTime
new parser is very simple - old school char by char state machine :) but it works

Thanks!

https://github.com/DCMTK/dcmtk/pull/17

Files

Download all files
github_pr17.patch (15.3 KB) github_pr17.patch GitHub pull request #17 Jörg Riesmeier, 2019-08-07 14:52
2ec7d368c.diff (18 KB) 2ec7d368c.diff Harald Roesen, 2026-02-12 11:45
2ec7d368c_v2.diff (18 KB) 2ec7d368c_v2.diff Harald Roesen, 2026-02-12 15:23
2ec7d368c_v3.diff (19.4 KB) 2ec7d368c_v3.diff Harald Roesen, 2026-02-16 14:42

Related issues 2 (1 open — 1 closed)

Related to DCMTK - Feature #1127: OFTime should distinguish between presence and absence of timezone informationClosedJörg Riesmeier2024-06-18

Actions
Blocks DCMTK - Feature #1128: dcmsr should support the fractional part of a second in its own XML formatNewJörg Riesmeier2024-06-19

Actions
Actions
Copy link
 #2

Updated by Jörg Riesmeier over 1 year ago

  • Related to Feature #1127: OFTime should distinguish between presence and absence of timezone information added
Actions
Copy link
 #3

Updated by Jörg Riesmeier over 1 year ago

  • Blocks Feature #1128: dcmsr should support the fractional part of a second in its own XML format added
Actions
Copy link
 #4

Updated by Harald Roesen about 1 month ago

  • Assignee set to Harald Roesen
Actions
Copy link
 #5

Updated by Harald Roesen 10 days ago

Please take a look at suggested changes in attached 'diff'.

Actions
Copy link
 #6

Updated by Harald Roesen 10 days ago

Updated suggested changes.

Actions
Copy link
 #7

Updated by Harald Roesen 6 days ago

Updated suggested changes.

Actions
Copy link
 #8

Updated by Harald Roesen 5 days ago

  • Status changed from Feedback to Closed

Closed with commit f3a0a2790.

Actions
Copy link

Also available in: Atom PDF