csharp/EPAM.Deltix.HdTime/Convert.cs (12 lines): - line 67: return x < 0 ? Int64.MinValue : Int64.MaxValue; // TODO: Bit operations possible - line 72: // TODO: could be implemented better - line 228: // TODO: !InRange(x, Max, Min); - line 343: // TODO: Delete unused - line 346: // // TODO: Better/faster check possible, but this will do for now - line 396: // TODO: Range check wrong? - line 438: private static Int64 ToMinMax(Int64 x) => x < 0 ? Min : Max; // TODO: Bit operations possible - line 473: // TODO: Optimization opportunity: Add() - line 515: // TODO: Optimize, remove dependency from DateTime - line 595: // TODO: !InRange(x, Max, Min); - line 742: private static Int64 ToMinMax(Int64 x) => x < 0 ? Min : Max; // TODO: Bit operations possible - line 811: // TODO: Optimization opportunity: Add() csharp/EPAM.Deltix.HdTime/Parsers.cs (11 lines): - line 291: // TODO: Range check - line 328: // TODO: Rangecheck for length > 5 - line 362: // TODO: Range check - line 373: // TODO: Range check - line 384: // TODO: Range check - line 395: // TODO: Range check - line 406: // TODO: Range check - line 417: // TODO: Range check - line 598: // TODO: NOTE: The static storage for field format descriptors is maybe redundant and the necessity of these methods is questionable - line 706: // TODO: Month names are currently unsupported - line 847: value.ResetTs(); // TODO: Remove java/main/src/main/java/com/epam/deltix/hdtime/Parsers.java (9 lines): - line 242: // TODO: Range check - line 287: // TODO: Rangecheck for length > 5 - line 295: // TODO: Range check - line 303: // TODO: Range check - line 311: // TODO: Range check - line 319: // TODO: Range check - line 327: // TODO: Range check - line 335: // TODO: Range check - line 565: // TODO: Month names are currently unsupported csharp/EPAM.Deltix.HdTime/HdDateTime.cs (7 lines): - line 41: /// TODO: - line 75: // TODO: Optimization opportunity - line 97: // TODO: Check for local time? - line 108: // TODO: Check range? - line 161: // TODO: Test range checks? - line 287: // TODO: Optimization opportunity - line 978: // TODO: .. java/main/src/main/java/com/epam/deltix/hdtime/Formatters.java (5 lines): - line 45: // TODO: Fields cache is not used yet - line 502: // TODO: Temporary implementation, remove soon - line 769: // TODO: This code can now be simplified by moving methods/fields to Formatter instance - line 861: // TODO: This code can now be simplified by moving methods/fields to Formatter instance - line 879: // TODO: Later: compare performance, also vs different charbuffer impl csharp/EPAM.Deltix.HdTime/Formatters.cs (4 lines): - line 454: // TODO: Temporary implementation, remove soon - line 575: // TODO: NOTE: The static storage for field format descriptors is maybe redundant and the necessity of these methods is questionable - line 699: // TODO: This code can now be simplified by moving methods/fields to Formatter instance - line 793: // TODO: This code can now be simplified by moving methods/fields to Formatter instance csharp/EPAM.Deltix.HdTime/HdTimeSpan.cs (3 lines): - line 48: /// TODO: - line 357: // TODO: Discuss: Do we need to handle Min/Max constants here when creating from millis value? - line 1069: // TODO: java/main/src/main/java/com/epam/deltix/hdtime/HdDateTimeUtils.java (3 lines): - line 72: // TODO: Min/Max checks? - line 261: // TODO: Less/Greater? Before/After ? Earlier/Later ? - line 299: * TODO: This looks misleading. Rounding for HdDateTime and rounding for HdTimeSpan java/main/src/main/java/com/epam/deltix/hdtime/Convert.java (2 lines): - line 371: calendar.clear(); // TODO: Slow - line 657: private static final double F_MAX = (double)MAX; // TODO: Verify vs integer values java/main/src/main/java/com/epam/deltix/hdtime/HdTimeSpanUtils.java (2 lines): - line 68: // TODO: Validate input parameters - line 314: * TODO: This looks misleading. Rounding for HdDateTime and rounding for HdTimeSpan csharp/EPAM.Deltix.HdTime/DateConverter.cs (2 lines): - line 42: // TODO: - line 72: // TODO: May silently overflow for certain big values java/main/src/main/java/com/epam/deltix/hdtime/HdTimeSpan.java (1 line): - line 443: * TODO: This looks misleading. Rounding for HdDateTime and rounding for HdTimeSpan should behave differently csharp/EPAM.Deltix.HdTime/FormatField.cs (1 line): - line 31: // TODO: Check returned value for different field types java/main/src/main/java/com/epam/deltix/hdtime/FormatField.java (1 line): - line 22: // NOTE: TODO: Possible that we define Field as Functional Interface, just to test the performance difference