fix(timing): Use i64 instead of c_long for Windows compatibility

The Rust FFI functions were using c_long for PTS/FTS timestamps, but:
- C code uses LLONG (int64_t, 64 bits on all platforms)
- Rust c_long is 32 bits on Windows, 64 bits on Linux

This caused timestamp truncation on Windows when PTS values exceeded
2^31 (~24 days at 90kHz), resulting in wrong subtitle timestamps.

For example, a file with Min PTS of 23:50:45 (7,726,090,500 ticks)
would have its PTS truncated, breaking the teletext delta calculation
that normalizes timestamps to start at 0.

Changes:
- ccxr_add_current_pts: pts parameter i64
- ccxr_set_current_pts: pts parameter i64
- ccxr_get_fts: return type i64
- ccxr_get_visible_end: return type i64
- ccxr_get_visible_start: return type i64
- ccxr_get_fts_max: return type i64
- ccxr_print_mstime_static: mstime parameter i64
- fts_at_gop_start: extern static i64

Fixes tests 18 and 19 on Windows CI which showed raw PTS timestamps
(23:50:46) instead of normalized timestamps (00:00:00).

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>

Author: cfsmp3

src/rust/src/decoder/output.rs

@@ -114,9 +114,12 @@ pub fn write_char(sym: &dtvcc_symbol, buf: &mut Vec<u8>, use_utf16: bool) {
 /// These are fixed-width 16-bit encodings where even ASCII needs 2 bytes.
 pub fn is_utf16_charset(charset: &str) -> bool {
     let upper = charset.to_uppercase();
-    upper.contains("UTF-16") || upper.contains("UTF16") ||
-    upper.contains("UCS-2") || upper.contains("UCS2") ||
-    upper.contains("UTF_16") || upper.contains("UCS_2")
+    upper.contains("UTF-16")
+        || upper.contains("UTF16")
+        || upper.contains("UCS-2")
+        || upper.contains("UCS2")
+        || upper.contains("UTF_16")
+        || upper.contains("UCS_2")
 }
 
 /// Convert from CEA-708 color representation to hex code

src/rust/src/es/pic.rs

@@ -277,12 +277,8 @@ pub unsafe fn read_pic_info(
     // set_fts() is not called for each picture when use_gop_as_pts == 1.
     if ccx_options.use_gop_as_pts == 1 {
         // Calculate current FTS based on GOP start time + frame offset
-        // Cast fts_at_gop_start to i64 for cross-platform compatibility (c_long is i32 on Windows)
         let frame_offset_ms = (dec_ctx.frames_since_last_gop as f64 * 1000.0 / current_fps) as i64;
-        #[allow(clippy::unnecessary_cast)]
-        {
-            (*dec_ctx.timing).fts_now = (fts_at_gop_start as i64) + frame_offset_ms;
-        }
+        (*dec_ctx.timing).fts_now = fts_at_gop_start + frame_offset_ms;
 
         // Update fts_max if needed
         if (*dec_ctx.timing).fts_now > (*dec_ctx.timing).fts_max {

src/rust/src/lib.rs

@@ -51,7 +51,7 @@ use std::os::raw::{c_uchar, c_void};
 use std::{
     ffi::CStr,
     io::Write,
-    os::raw::{c_char, c_double, c_int, c_long, c_uint},
+    os::raw::{c_char, c_double, c_int, c_uint},
 };
 
 // Mock data for rust unit tests
@@ -67,7 +67,7 @@ cfg_if! {
 
         static mut frames_since_ref_time: c_int = 0;
         static mut total_frames_count: c_uint = 0;
-        static mut fts_at_gop_start: c_long = 0;
+        static mut fts_at_gop_start: i64 = 0;
         static mut gop_rollover: c_int = 0;
         static mut pts_big_change: c_uint = 0;
 
@@ -144,7 +144,7 @@ extern "C" {
     static mut total_frames_count: c_uint;
     static mut gop_time: gop_time_code;
     static mut first_gop_time: gop_time_code;
-    static mut fts_at_gop_start: c_long;
+    static mut fts_at_gop_start: i64;
     static mut gop_rollover: c_int;
     static mut ccx_common_timing_settings: ccx_common_timing_settings_t;
     static mut capitalization_list: word_list;

src/rust/src/libccxr_exports/time.rs

@@ -1,7 +1,7 @@
 #![allow(clippy::useless_conversion)]
 
 use std::convert::TryInto;
-use std::ffi::{c_char, c_int, c_long, CStr};
+use std::ffi::{c_char, c_int, CStr};
 
 use crate::{
     bindings::*, cb_708, cb_field1, cb_field2, ccx_common_timing_settings as timing_settings,
@@ -330,7 +330,7 @@ unsafe fn write_back_from_timing_info() {
         .unwrap_or(0);
     gop_time = write_gop_time_code(timing_info.gop_time);
     first_gop_time = write_gop_time_code(timing_info.first_gop_time);
-    fts_at_gop_start = timing_info.fts_at_gop_start.millis() as c_long;
+    fts_at_gop_start = timing_info.fts_at_gop_start.millis();
     gop_rollover = if timing_info.gop_rollover { 1 } else { 0 };
     timing_settings.disable_sync_check = if timing_info.timing_settings.disable_sync_check {
         1
@@ -412,7 +412,7 @@ pub unsafe fn write_gop_time_code(g: Option<GopTimeCode>) -> gop_time_code {
 ///
 /// `ctx` must not be null.
 #[no_mangle]
-pub unsafe extern "C" fn ccxr_add_current_pts(ctx: *mut ccx_common_timing_ctx, pts: c_long) {
+pub unsafe extern "C" fn ccxr_add_current_pts(ctx: *mut ccx_common_timing_ctx, pts: i64) {
     apply_timing_info();
     let mut context = generate_timing_context(ctx);
 
@@ -428,7 +428,7 @@ pub unsafe extern "C" fn ccxr_add_current_pts(ctx: *mut ccx_common_timing_ctx, p
 ///
 /// `ctx` must not be null.
 #[no_mangle]
-pub unsafe extern "C" fn ccxr_set_current_pts(ctx: *mut ccx_common_timing_ctx, pts: c_long) {
+pub unsafe extern "C" fn ccxr_set_current_pts(ctx: *mut ccx_common_timing_ctx, pts: i64) {
     apply_timing_info();
     let mut context = generate_timing_context(ctx);
 
@@ -469,7 +469,7 @@ pub unsafe extern "C" fn ccxr_set_fts(ctx: *mut ccx_common_timing_ctx) -> c_int
 pub unsafe extern "C" fn ccxr_get_fts(
     ctx: *mut ccx_common_timing_ctx,
     current_field: c_int,
-) -> c_long {
+) -> i64 {
     apply_timing_info();
     let mut context = generate_timing_context(ctx);
 
@@ -485,7 +485,7 @@ pub unsafe extern "C" fn ccxr_get_fts(
     write_back_to_common_timing_ctx(ctx, &context);
     write_back_from_timing_info();
 
-    ans.millis().try_into().unwrap_or(0)
+    ans.millis()
 }
 
 /// Rust equivalent for `get_visible_end` function in C. Uses C-native types as input and output.
@@ -503,7 +503,7 @@ pub unsafe extern "C" fn ccxr_get_fts(
 pub unsafe extern "C" fn ccxr_get_visible_end(
     ctx: *mut ccx_common_timing_ctx,
     _current_field: c_int,
-) -> c_long {
+) -> i64 {
     apply_timing_info();
     let mut context = generate_timing_context(ctx);
 
@@ -518,7 +518,7 @@ pub unsafe extern "C" fn ccxr_get_visible_end(
     write_back_to_common_timing_ctx(ctx, &context);
     write_back_from_timing_info();
 
-    fts as c_long
+    fts
 }
 
 /// Rust equivalent for `get_visible_start` function in C. Uses C-native types as input and output.
@@ -537,7 +537,7 @@ pub unsafe extern "C" fn ccxr_get_visible_end(
 pub unsafe extern "C" fn ccxr_get_visible_start(
     ctx: *mut ccx_common_timing_ctx,
     _current_field: c_int,
-) -> c_long {
+) -> i64 {
     apply_timing_info();
     let context = generate_timing_context(ctx);
 
@@ -554,7 +554,7 @@ pub unsafe extern "C" fn ccxr_get_visible_start(
     write_back_to_common_timing_ctx(ctx, &context);
     write_back_from_timing_info();
 
-    fts as c_long
+    fts
 }
 
 /// Rust equivalent for `get_fts_max` function in C. Uses C-native types as input and output.
@@ -563,7 +563,7 @@ pub unsafe extern "C" fn ccxr_get_visible_start(
 ///
 /// `ctx` must not be null.
 #[no_mangle]
-pub unsafe extern "C" fn ccxr_get_fts_max(ctx: *mut ccx_common_timing_ctx) -> c_long {
+pub unsafe extern "C" fn ccxr_get_fts_max(ctx: *mut ccx_common_timing_ctx) -> i64 {
     apply_timing_info();
     let mut context = generate_timing_context(ctx);
 
@@ -572,7 +572,7 @@ pub unsafe extern "C" fn ccxr_get_fts_max(ctx: *mut ccx_common_timing_ctx) -> c_
     write_back_to_common_timing_ctx(ctx, &context);
     write_back_from_timing_info();
 
-    ans.millis().try_into().unwrap_or(0)
+    ans.millis()
 }
 
 /// Rust equivalent for `print_mstime_static` function in C. Uses C-native types as input and output.
@@ -581,8 +581,8 @@ pub unsafe extern "C" fn ccxr_get_fts_max(ctx: *mut ccx_common_timing_ctx) -> c_
 ///
 /// `buf` must not be null. It must have sufficient length to hold the time in string form.
 #[no_mangle]
-pub unsafe extern "C" fn ccxr_print_mstime_static(mstime: c_long, buf: *mut c_char) -> *mut c_char {
-    let time = Timestamp::from_millis(mstime.into());
+pub unsafe extern "C" fn ccxr_print_mstime_static(mstime: i64, buf: *mut c_char) -> *mut c_char {
+    let time = Timestamp::from_millis(mstime);
     let ans = c::print_mstime_static(time, ':');
     write_string_into_pointer(buf, &ans);
     buf