Support Board
Date/Time: Tue, 27 Oct 2026 03:14:07 +0000
[minor] v2978 header changes
View Count: 15
| [2026-10-27 01:31:57] |
| User719512 - Posts: 516 |
|
Hi Sierra Chart Engineering, Picked up v2978 and looked at changes and noticed: DEPRECATED("The HMS_TIME is deprecated. Use SCDateTime:HMSToTime instead.") should have two :: not one : DEPRECATED("The HMS_TIME is deprecated. Use SCDateTime::HMSToTime instead.") to possibly prevent the vibe coders from getting compile errors. scdatetime.h declares SCDateTime& AddNanoseconds(const std::uint64_t Nanoseconds); but Function definition for 'AddNanoseconds' not found. While we are discussing header files, perhaps you'd be open to changing GetDateTimeOfLastTradeRecord to remove the warning about Warning C6295 Ill-defined for-loop. Loop executes infinitely. scstructures.h 422
This is possible refactor: SCDateTimeMS GetDateTimeOfLastTradeRecord() { unsigned int RecordIndex = Size(); while (RecordIndex > 0) { --RecordIndex; if (IsRecordIndexTrade(RecordIndex)) return operator[](RecordIndex).DateTime; } return 0.0; } |
To post a message in this thread, you need to log in with your Sierra Chart account: