28b79bf81b
Merge pull request 'C2026-053' (#813) from C2026-053_restore_only_Proc_section_tabs into Development
1e0e3a7114
Merge pull request 'B2026-074 Fixed issue where PROMS would hang when opening multiple saved Word section tabs. Also, we now save and restore to the step location of saved procedure editor sections.' (#812) from B2026-074_restore_proc_tabs into Development
7333247b52
B2026-074 Fixed issue where PROMS would hang when opening multiple saved Word section tabs. Also, we now save and restore to the step location of saved procedure editor sections.
e16b3262d9
Merge pull request 'C2026-041-invisible-position-changes-as-differences' (#807) from C2026-041-invisible-position-changes-as-differences into Development
1e0e3a7114
Merge pull request 'B2026-074 Fixed issue where PROMS would hang when opening multiple saved Word section tabs. Also, we now save and restore to the step location of saved procedure editor sections.' (#812) from B2026-074_restore_proc_tabs into Development
7333247b52
B2026-074 Fixed issue where PROMS would hang when opening multiple saved Word section tabs. Also, we now save and restore to the step location of saved procedure editor sections.
Looks good. Ready for QA.
Looks like there is still some commented out code and there was an added parameter that is unused.
foundFileFg is an added paramater but it is not used - if not used it shoul be removed.
foundFileFg is an added paramater but it is not used - if not used it shoul be removed.
This is still commented out code - commented out code should be removed.
private void cbUseTruncatedData_Click(object sender, EventArgs e)
This seems unnecessary as it is setting a global variable TruncatedDataFg - which is never used.
Sorry noticed one additional thing --- TruncatedDataFg was added as a global variable and set but is never used as this is passed to functions and used locally --- as such, shouldn't also be a global variable / be in memory and not used.
Wait this is both a global variable --- but it looks like it is passed as a paramater to functions --- if it is being passed as a paramaeter used locally in functions, why is it also a global variable?