Remove VCD interactive mode

CHANGELOG.md

@@ -6,6 +6,16 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
 ## [Unreleased]
 
+### Current limitations compared to GTKWave LTS 3.3
+
+Some features have been removed in this version of GTKWave to make the codebase
+easier to refactor. These features might be restored in future versions of GTKWave,
+if there is enough interest by users. Please notify us on GitHub if you require any
+of these features.
+
+- Removed the interactive VCD mode.
+- Removed support for LXT, LXT2, VZT, and AET2 formats.
+
 ### Changed
 
 - Merged in major GUI widget refactoring.
@@ -27,7 +37,6 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 ### Removed
 
 - Removed support for GTK 2.
-- Removed support for LXT, LXT2, VZT, and AET2 formats.
 - Removed "Signal Search Hierarchy" dialog.
 - Removed setting to disable the "Alternate Wheel Mode".
 - Removed legacy VCD loader.

man/gtkwave.1

@@ -123,12 +123,6 @@ Set gtk-application-prefer-dark-theme = TRUE.
 \fB\-7\fR,\fB\-\-saveonexit\fR
 At exit, a requester is brought up to prompt user to write a save file.  Canceling the requester prevents from writing the file.
 .TP
-\fB\-I\fR,\fB\-\-interactive\fR
-Specifies that "interactive" VCD mode is to be used which allows a viewer to navigate a VCD trace while GTKWave is processing the VCD file.
-When this option is used, the filename is overloaded such that it is the hexadecimal value for the shared memory ID of a writer.
-Note that the shared memory ID can be passed straight from stdin by using the \fB\-\-vcd\fR option; see the manpage for
-\fIshmidcat\fP(1) for more details.
-.TP
 \fB\-g\fR,\fB\-\-giga\fR
 Specifies that the viewer should use gigabyte mempacking when recoding (possibly slower).  This is equivalent to setting
 the vlist_spill and vlist_prepack flags in the rc file.

man/gtkwaverc.5

@@ -25,9 +25,6 @@ This allows another character in addition to the hier_delimeter to be used to de
 Specifies how many overlapping analog segments can be drawn for a given X position onscreen.  (Default: 20)  If there are
 gaps in analog traces, this value is too low.
 .TP
-\fBappend_vcd_hier\fR <\fIvalue\fP>
-Allows the specification of a prefix hierarchy for VCD files. This can be done in "pieces," so that multiple layers of hierarchy are prepended to symbol names with the most significant addition occurring first (see .gtkwaverc in the  examples/vcd directory). The intended use of this is to have the ability to add "project" prefixes which allow easier selection of everything from the tree hierarchy.
-.TP 
 \fBautocoalesce\fR <\fIvalue\fP>
 A nonzero value enables autocoalescing of VCD vectors when applicable. This may be toggled dynamically during wave viewer usage.
 .TP 

man/meson.build

@@ -6,7 +6,6 @@ install_man(
     'lxt2miner.1',
     'lxt2vcd.1',
     'rtlbrowse.1',
-    'shmidcat.1',
     'twinwave.1',
     'vcd2fst.1',
     'vcd2lxt.1',

src/currenttime.h

@@ -60,7 +60,6 @@ void exponent_to_time_scale(signed char scale);
 #include "timeentry.h"
 #include "tree.h"
 #include "treesearch.h"
-#include "vcd_partial.h"
 #include "wavewindow.h"
 #include "zoombuttons.h"
 

src/globals.c

@@ -727,45 +727,6 @@ static const struct Global globals_base_values = {
     NULL, /* he_curr_vcd_c_1 506 */
     NULL, /* he_fini */
 
-    /*
-     * vcd_partial.c
-     */
-    0, /* vcdbyteno_vcd_partial_c_2 516 */
-    0, /* error_count_vcd_partial_c_2 517 */
-    0, /* header_over_vcd_partial_c_2 518 */
-    -1, /* start_time_vcd_partial_c_2 520 */
-    -1, /* end_time_vcd_partial_c_2 521 */
-    -1, /* current_time_vcd_partial_c_2 522 */
-    0, /* num_glitches_vcd_partial_c_3 523 */
-    0, /* num_glitch_regions_vcd_partial_c_3 524 */
-    NULL, /* pv_vcd_partial_c_2 525 */
-    NULL, /* rootv */
-    NULL, /* vcdbuf_vcd_partial_c_2 526 */
-    NULL, /* vst */
-    NULL, /* vend */
-    NULL, /* consume_ptr_vcd_partial_c_1 527 */
-    NULL, /* buf_vcd_partial_c_2 528 */
-    100000, /* consume_countdown_vcd_partial_c_1 529 */
-    1024, /* T_MAX_STR_vcd_partial_c_2 531 */
-    NULL, /* yytext_vcd_partial_c_2 532 */
-    0, /* yylen_vcd_partial_c_2 533 */
-    0, /* yylen_cache */
-    NULL, /* vcdsymroot_vcd_partial_c_2 534 */
-    NULL, /* vcdsymcurr */
-    NULL, /* sorted_vcd_partial_c_2 535 */
-    NULL, /* indexed_vcd_partial_c_2 536 */
-    0, /* numsyms_vcd_partial_c_2 538 */
-    ~0, /* vcd_minid_vcd_partial_c_2 540 */
-    0, /* vcd_maxid_vcd_partial_c_2 541 */
-    0, /* err_vcd_partial_c_2 542 */
-    NULL, /* varsplit_vcd_partial_c_2 543 */
-    NULL, /* vsplitcurr */
-    0, /* var_prevch_vcd_partial_c_2 544 */
-    0, /* timeset_vcd_partial_c_1 547 */
-    0, /* vcd_hash_max_partial */
-    0, /* vcd_hash_kill_partial */
-    NULL, /* sym_chain_partial */
-
     /*
      * vcd_recoder.c
      */

src/globals.h

@@ -707,45 +707,6 @@ struct Global
     GwHistEnt *he_curr_vcd_c_1; /* from vcd.c 543 */
     GwHistEnt *he_fini_vcd_c_1; /* from vcd.c 544 */
 
-    /*
-     * vcd_partial.c
-     */
-    off_t vcdbyteno_vcd_partial_c_2; /* from vcd_partial.c 555 */
-    int error_count_vcd_partial_c_2; /* from vcd_partial.c 556 */
-    int header_over_vcd_partial_c_2; /* from vcd_partial.c 557 */
-    GwTime start_time_vcd_partial_c_2; /* from vcd_partial.c 559 */
-    GwTime end_time_vcd_partial_c_2; /* from vcd_partial.c 560 */
-    GwTime current_time_vcd_partial_c_2; /* from vcd_partial.c 561 */
-    int num_glitches_vcd_partial_c_3; /* from vcd_partial.c 562 */
-    int num_glitch_regions_vcd_partial_c_3; /* from vcd_partial.c 563 */
-    struct vcdsymbol *pv_vcd_partial_c_2; /* from vcd_partial.c 564 */
-    struct vcdsymbol *rootv_vcd_partial_c_2; /* from vcd_partial.c 565 */
-    char *vcdbuf_vcd_partial_c_2; /* from vcd_partial.c 566 */
-    char *vst_vcd_partial_c_2; /* from vcd_partial.c 567 */
-    char *vend_vcd_partial_c_2; /* from vcd_partial.c 568 */
-    char *consume_ptr_vcd_partial_c_1; /* from vcd_partial.c 569 */
-    char *buf_vcd_partial_c_2; /* from vcd_partial.c 570 */
-    int consume_countdown_vcd_partial_c_1; /* from vcd_partial.c 571 */
-    int T_MAX_STR_vcd_partial_c_2; /* from vcd_partial.c 573 */
-    char *yytext_vcd_partial_c_2; /* from vcd_partial.c 574 */
-    int yylen_vcd_partial_c_2; /* from vcd_partial.c 575 */
-    int yylen_cache_vcd_partial_c_2; /* from vcd_partial.c 576 */
-    struct vcdsymbol *vcdsymroot_vcd_partial_c_2; /* from vcd_partial.c 577 */
-    struct vcdsymbol *vcdsymcurr_vcd_partial_c_2; /* from vcd_partial.c 578 */
-    struct vcdsymbol **sorted_vcd_partial_c_2; /* from vcd_partial.c 579 */
-    struct vcdsymbol **indexed_vcd_partial_c_2; /* from vcd_partial.c 580 */
-    int numsyms_vcd_partial_c_2; /* from vcd_partial.c 582 */
-    unsigned int vcd_minid_vcd_partial_c_2; /* from vcd_partial.c 584 */
-    unsigned int vcd_maxid_vcd_partial_c_2; /* from vcd_partial.c 585 */
-    int err_vcd_partial_c_2; /* from vcd_partial.c 586 */
-    char *varsplit_vcd_partial_c_2; /* from vcd_partial.c 587 */
-    char *vsplitcurr_vcd_partial_c_2; /* from vcd_partial.c 588 */
-    int var_prevch_vcd_partial_c_2; /* from vcd_partial.c 589 */
-    int timeset_vcd_partial_c_1; /* from vcd_partial.c 592 */
-    unsigned int vcd_hash_max_partial; 
-    int vcd_hash_kill_partial;
-    GSList *sym_chain_partial;
-
     /*
      * vcd_recoder.c
      */

src/helpers/meson.build

@@ -4,7 +4,6 @@ helpers = [
     'fstminer',
     'lxt2miner',
     'lxt2vcd',
-    'shmidcat',
     'vcd2fst',
     'vcd2lxt',
     'vcd2lxt2',