]>
Commit | Line | Data |
---|---|---|
21b7d176 ER |
1 | To: vim_dev@googlegroups.com |
2 | Subject: Patch 7.3.108 | |
3 | Fcc: outbox | |
4 | From: Bram Moolenaar <Bram@moolenaar.net> | |
5 | Mime-Version: 1.0 | |
6 | Content-Type: text/plain; charset=UTF-8 | |
7 | Content-Transfer-Encoding: 8bit | |
8 | ------------ | |
9 | ||
10 | Patch 7.3.108 | |
11 | Problem: Useless check for NULL when calling vim_free(). | |
12 | Solution: Remove the check. (Dominique Pelle) | |
13 | Files: src/eval.c, src/ex_cmds.c, src/os_win32.c | |
14 | ||
15 | ||
16 | *** ../vim-7.3.107/src/eval.c 2011-01-22 01:13:33.000000000 +0100 | |
17 | --- src/eval.c 2011-01-30 21:37:53.000000000 +0100 | |
18 | *************** | |
19 | *** 5106,5114 **** | |
20 | else | |
21 | ret = OK; | |
22 | } | |
23 | ! | |
24 | ! if (alias != NULL) | |
25 | ! vim_free(alias); | |
26 | } | |
27 | ||
28 | *arg = skipwhite(*arg); | |
29 | --- 5106,5112 ---- | |
30 | else | |
31 | ret = OK; | |
32 | } | |
33 | ! vim_free(alias); | |
34 | } | |
35 | ||
36 | *arg = skipwhite(*arg); | |
37 | *************** | |
38 | *** 19807,19813 **** | |
39 | EMSG2(_("E704: Funcref variable name must start with a capital: %s"), name); | |
40 | return; | |
41 | } | |
42 | ! /* Don't allow hiding a function. When "v" is not NULL we migth be | |
43 | * assigning another function to the same var, the type is checked | |
44 | * below. */ | |
45 | if (v == NULL && function_exists(name)) | |
46 | --- 19805,19811 ---- | |
47 | EMSG2(_("E704: Funcref variable name must start with a capital: %s"), name); | |
48 | return; | |
49 | } | |
50 | ! /* Don't allow hiding a function. When "v" is not NULL we might be | |
51 | * assigning another function to the same var, the type is checked | |
52 | * below. */ | |
53 | if (v == NULL && function_exists(name)) | |
54 | *** ../vim-7.3.107/src/ex_cmds.c 2010-12-17 18:06:00.000000000 +0100 | |
55 | --- src/ex_cmds.c 2011-01-30 21:37:53.000000000 +0100 | |
56 | *************** | |
57 | *** 5412,5418 **** | |
58 | vir_T *virp; | |
59 | int force; | |
60 | { | |
61 | ! if (old_sub != NULL && force) | |
62 | vim_free(old_sub); | |
63 | if (force || old_sub == NULL) | |
64 | old_sub = viminfo_readstring(virp, 1, TRUE); | |
65 | --- 5412,5418 ---- | |
66 | vir_T *virp; | |
67 | int force; | |
68 | { | |
69 | ! if (force) | |
70 | vim_free(old_sub); | |
71 | if (force || old_sub == NULL) | |
72 | old_sub = viminfo_readstring(virp, 1, TRUE); | |
73 | *** ../vim-7.3.107/src/os_win32.c 2010-12-17 20:23:56.000000000 +0100 | |
74 | --- src/os_win32.c 2011-01-30 21:37:53.000000000 +0100 | |
75 | *************** | |
76 | *** 1886,1893 **** | |
77 | cb->BufferSize.X = cb->Info.dwSize.X; | |
78 | cb->BufferSize.Y = cb->Info.dwSize.Y; | |
79 | NumCells = cb->BufferSize.X * cb->BufferSize.Y; | |
80 | ! if (cb->Buffer != NULL) | |
81 | ! vim_free(cb->Buffer); | |
82 | cb->Buffer = (PCHAR_INFO)alloc(NumCells * sizeof(CHAR_INFO)); | |
83 | if (cb->Buffer == NULL) | |
84 | return FALSE; | |
85 | --- 1886,1892 ---- | |
86 | cb->BufferSize.X = cb->Info.dwSize.X; | |
87 | cb->BufferSize.Y = cb->Info.dwSize.Y; | |
88 | NumCells = cb->BufferSize.X * cb->BufferSize.Y; | |
89 | ! vim_free(cb->Buffer); | |
90 | cb->Buffer = (PCHAR_INFO)alloc(NumCells * sizeof(CHAR_INFO)); | |
91 | if (cb->Buffer == NULL) | |
92 | return FALSE; | |
93 | *** ../vim-7.3.107/src/version.c 2011-01-22 21:25:07.000000000 +0100 | |
94 | --- src/version.c 2011-02-01 13:47:07.000000000 +0100 | |
95 | *************** | |
96 | *** 716,717 **** | |
97 | --- 716,719 ---- | |
98 | { /* Add new patch number below this line */ | |
99 | + /**/ | |
100 | + 108, | |
101 | /**/ | |
102 | ||
103 | -- | |
104 | hundred-and-one symptoms of being an internet addict: | |
105 | 173. You keep tracking down the email addresses of all your friends | |
106 | (even childhood friends). | |
107 | ||
108 | /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net \\\ | |
109 | /// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\ | |
110 | \\\ an exciting new programming language -- http://www.Zimbu.org /// | |
111 | \\\ help me help AIDS victims -- http://ICCF-Holland.org /// |