Support Board
Date/Time: Tue, 27 Oct 2026 03:46:11 +0000
[minor] v2978 header changes
View Count: 20
| [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; } |
| [2026-10-27 03:46:06] |
| Sierra_Chart Engineering - Posts: 25291 |
|
This is in the process of being removed along with other deprecated functions: DEPRECATED("The HMS_TIME is deprecated. Use SCDateTime:HMSToTime instead.") So it is no longer relevant and thank you for pointing that out. The nanosecond function, we will remove that. This has already been resolved before this report: GetDateTimeOfLastTradeRecord Sierra Chart Support - Engineering Level Your definitive source for support. Other responses are from users. Try to keep your questions brief and to the point. Be aware of support policy: https://www.sierrachart.com/index.php?l=PostingInformation.php#GeneralInformation For the most reliable, advanced, and zero cost futures order routing, use the Teton service: Sierra Chart Teton Futures Order Routing |
To post a message in this thread, you need to log in with your Sierra Chart account: