Patch 8.2.3740

5 views
Skip to first unread message

Bram Moolenaar

unread,
Dec 5, 2021, 6:37:11 AM12/5/21
to vim...@googlegroups.com

Patch 8.2.3740
Problem: Memory left allocated on exit when using Tcl.
Solution: Call Tcl_Finalize().
Files: src/if_tcl.c, src/proto/if_tcl.pro, src/alloc.c


*** ../vim-8.2.3739/src/if_tcl.c 2021-07-24 12:57:25.481296527 +0100
--- src/if_tcl.c 2021-12-05 11:31:53.800453254 +0000
***************
*** 248,253 ****
--- 248,261 ----
#endif
}

+ #if defined(EXITFREE) || defined(PROTO)
+ void
+ vim_tcl_finalize(void)
+ {
+ Tcl_Finalize();
+ }
+ #endif
+
#if defined(DYNAMIC_TCL) || defined(PROTO)

static int stubs_initialized = FALSE;
*** ../vim-8.2.3739/src/proto/if_tcl.pro 2019-12-12 11:55:41.000000000 +0000
--- src/proto/if_tcl.pro 2021-12-05 11:31:57.728438425 +0000
***************
*** 1,5 ****
--- 1,6 ----
/* if_tcl.c */
void vim_tcl_init(char *arg);
+ void vim_tcl_finalize(void);
int tcl_enabled(int verbose);
void tcl_end(void);
void ex_tcl(exarg_T *eap);
*** ../vim-8.2.3739/src/alloc.c 2021-12-04 14:02:25.457024636 +0000
--- src/alloc.c 2021-12-05 11:30:51.164690006 +0000
***************
*** 574,579 ****
--- 574,582 ----
# ifdef FEAT_GUI_GTK
gui_mch_free_all();
# endif
+ # ifdef FEAT_TCL
+ vim_tcl_finalize();
+ # endif
clear_hl_tables();

vim_free(IObuff);
*** ../vim-8.2.3739/src/version.c 2021-12-04 22:03:30.720421318 +0000
--- src/version.c 2021-12-05 11:31:48.628472810 +0000
***************
*** 755,756 ****
--- 755,758 ----
{ /* Add new patch number below this line */
+ /**/
+ 3740,
/**/

--
Article in the first Free Software Magazine: "Bram Moolenaar studied
electrical engineering at the Technical University of Delft and
graduated in 1985 on a multi-processor Unix architecture."
Response by "dimator": Could the school not afford a proper
stage for the ceremony?

/// Bram Moolenaar -- Br...@Moolenaar.net -- http://www.Moolenaar.net \\\
/// \\\
\\\ sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
Reply all
Reply to author
Forward
0 new messages