]> git.ipfire.org Git - thirdparty/vim.git/commitdiff
patch 8.0.1680: memory allocated by libvterm is not profiled v8.0.1680
authorBram Moolenaar <Bram@vim.org>
Tue, 10 Apr 2018 10:04:27 +0000 (12:04 +0200)
committerBram Moolenaar <Bram@vim.org>
Tue, 10 Apr 2018 10:04:27 +0000 (12:04 +0200)
Problem:    Memory allocated by libvterm does not show up in profile.
Solution:   Pass allocater functions to vterm_new().

src/terminal.c
src/version.c

index 1d81d8db5681bb7ee83aaf15404f767c13d2464f..69b3306a412a15d7a803c0e1615ca44082816be7 100644 (file)
  *
  * TODO:
  * - Add a way to set the 16 ANSI colors, to be used for 'termguicolors' and in
- *   the GUI.
+ *   the GUI. #2747
  * - Win32: Make terminal used for :!cmd in the GUI work better.  Allow for
  *   redirection.  Probably in call to channel_set_pipes().
  * - implement term_setsize()
  * - Copy text in the vterm to the Vim buffer once in a while, so that
  *   completion works.
- * - Adding WinBar to terminal window doesn't display, text isn't shifted down.
- *   a job that uses 16 colors while Vim is using > 256.
  * - in GUI vertical split causes problems.  Cursor is flickering. (Hirohito
  *   Higashi, 2017 Sep 19)
  * - after resizing windows overlap. (Boris Staletic, #2164)
@@ -3329,6 +3327,26 @@ static VTermParserCallbacks parser_fallbacks = {
   NULL         /* resize */
 };
 
+/*
+ * Use Vim's allocation functions for vterm so profiling works.
+ */
+    static void *
+vterm_malloc(size_t size, void *data UNUSED)
+{
+    return alloc_clear(size);
+}
+
+    static void
+vterm_memfree(void *ptr, void *data UNUSED)
+{
+    vim_free(ptr);
+}
+
+static VTermAllocatorFunctions vterm_allocator = {
+  &vterm_malloc,
+  &vterm_memfree
+};
+
 /*
  * Create a new vterm and initialize it.
  */
@@ -3340,7 +3358,7 @@ create_vterm(term_T *term, int rows, int cols)
     VTermState     *state;
     VTermValue     value;
 
-    vterm = vterm_new(rows, cols);
+    vterm = vterm_new_with_allocator(rows, cols, &vterm_allocator, NULL);
     term->tl_vterm = vterm;
     screen = vterm_obtain_screen(vterm);
     vterm_screen_set_callbacks(screen, &screen_callbacks, term);
index a044f09bc1fff8bd681a1bf4234962272c66d1a4..f0a21b335808ada39a91e875cf04dec23bc53e09 100644 (file)
@@ -762,6 +762,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    1680,
 /**/
     1679,
 /**/