home *** CD-ROM | disk | FTP | other *** search
/ vim.ftp.fu-berlin.de / 2015-02-03.vim.ftp.fu-berlin.de.tar / vim.ftp.fu-berlin.de / patches / 7.3 / 7.3.943 < prev    next >
Encoding:
Internet Message Format  |  2013-05-11  |  6.7 KB
  1. To: vim_dev@googlegroups.com
  2. Subject: Patch 7.3.943
  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.943
  11. Problem:    Python: Negative indices were failing.
  12. Solution:   Fix negative indices. Add tests. (ZyX)
  13. Files:        src/if_py_both.h, src/if_python3.c, src/testdir/test86.in,
  14.         src/testdir/test86.ok, src/testdir/test87.in,
  15.         src/testdir/test87.ok
  16.  
  17.  
  18. *** ../vim-7.3.942/src/if_py_both.h    2013-05-12 20:36:09.000000000 +0200
  19. --- src/if_py_both.h    2013-05-12 21:10:03.000000000 +0200
  20. ***************
  21. *** 2394,2399 ****
  22. --- 2394,2402 ----
  23.       if (end == -1)
  24.       end = self->buf->b_ml.ml_line_count;
  25.   
  26. +     if (n < 0)
  27. +     n += end - start + 1;
  28. + 
  29.       if (n < 0 || n > end - start)
  30.       {
  31.       PyErr_SetString(PyExc_IndexError, _("line number out of range"));
  32. ***************
  33. *** 2441,2446 ****
  34. --- 2444,2452 ----
  35.       if (end == -1)
  36.       end = self->buf->b_ml.ml_line_count;
  37.   
  38. +     if (n < 0)
  39. +     n += end - start + 1;
  40. + 
  41.       if (n < 0 || n > end - start)
  42.       {
  43.       PyErr_SetString(PyExc_IndexError, _("line number out of range"));
  44. *** ../vim-7.3.942/src/if_python3.c    2013-05-12 20:36:09.000000000 +0200
  45. --- src/if_python3.c    2013-05-12 21:10:03.000000000 +0200
  46. ***************
  47. *** 1114,1120 ****
  48.           return NULL;
  49.   
  50.       if (PySlice_GetIndicesEx((PyObject *)idx,
  51. !           (Py_ssize_t)((BufferObject *)(self))->buf->b_ml.ml_line_count+1,
  52.             &start, &stop,
  53.             &step, &slicelen) < 0)
  54.       {
  55. --- 1114,1120 ----
  56.           return NULL;
  57.   
  58.       if (PySlice_GetIndicesEx((PyObject *)idx,
  59. !           (Py_ssize_t)((BufferObject *)(self))->buf->b_ml.ml_line_count,
  60.             &start, &stop,
  61.             &step, &slicelen) < 0)
  62.       {
  63. ***************
  64. *** 1146,1152 ****
  65.           return -1;
  66.   
  67.       if (PySlice_GetIndicesEx((PyObject *)idx,
  68. !           (Py_ssize_t)((BufferObject *)(self))->buf->b_ml.ml_line_count+1,
  69.             &start, &stop,
  70.             &step, &slicelen) < 0)
  71.       {
  72. --- 1146,1152 ----
  73.           return -1;
  74.   
  75.       if (PySlice_GetIndicesEx((PyObject *)idx,
  76. !           (Py_ssize_t)((BufferObject *)(self))->buf->b_ml.ml_line_count,
  77.             &start, &stop,
  78.             &step, &slicelen) < 0)
  79.       {
  80. *** ../vim-7.3.942/src/testdir/test86.in    2013-05-06 03:52:44.000000000 +0200
  81. --- src/testdir/test86.in    2013-05-12 21:11:43.000000000 +0200
  82. ***************
  83. *** 475,483 ****
  84.   :       endtry
  85.   :   endfor
  86.   :   call RecVars(oname)
  87. - endtry
  88.   :endfor
  89.   :only
  90.   :endfun
  91.   :"
  92.   :call Test()
  93. --- 475,524 ----
  94.   :       endtry
  95.   :   endfor
  96.   :   call RecVars(oname)
  97.   :endfor
  98.   :only
  99. + :"
  100. + :" Test buffer object
  101. + :vnew
  102. + :put ='First line'
  103. + :put ='Second line'
  104. + :put ='Third line'
  105. + :1 delete _
  106. + :py b=vim.current.buffer
  107. + :wincmd w
  108. + :mark a
  109. + py << EOF
  110. + cb = vim.current.buffer
  111. + # Tests BufferAppend and BufferItem
  112. + cb.append(b[0])
  113. + # Tests BufferSlice and BufferAssSlice
  114. + cb.append('abc') # Will be overwritten
  115. + cb[-1:] = b[:-2]
  116. + # Test BufferLength and BufferAssSlice
  117. + cb.append('def') # Will not be overwritten
  118. + cb[len(cb):] = b[:]
  119. + # Test BufferAssItem and BufferMark
  120. + cb.append('ghi') # Will be overwritten
  121. + cb[-1] = repr((len(cb) - cb.mark('a')[0], cb.mark('a')[1]))
  122. + # Test BufferRepr
  123. + cb.append(repr(cb) + repr(b))
  124. + # Modify foreign buffer
  125. + b.append('foo')
  126. + b[0]='bar'
  127. + b[0:0]=['baz']
  128. + vim.command('call append("$", getbufline(%i, 1, "$"))' % b.number)
  129. + # Test CheckBuffer
  130. + vim.command('bwipeout! ' + str(b.number))
  131. + for expr in ('b[1]','b[:] = ["A", "B"]','b[:]','b.append("abc")'):
  132. +     try:
  133. +         exec(expr)
  134. +     except vim.error:
  135. +         pass
  136. +     else:
  137. +         # Usually a SEGV here
  138. +         # Should not happen in any case
  139. +         cb.append('No exception for ' + expr)
  140. + EOF
  141.   :endfun
  142.   :"
  143.   :call Test()
  144. *** ../vim-7.3.942/src/testdir/test86.ok    2013-05-06 03:52:44.000000000 +0200
  145. --- src/testdir/test86.ok    2013-05-12 21:11:43.000000000 +0200
  146. ***************
  147. *** 306,308 ****
  148. --- 306,321 ----
  149.     G: '.,,'
  150.     W: 1:',,' 2:'.,,' 3:'.,,' 4:'.,,'
  151.     B: 1:',,' 2:'.,,' 3:'.,,' 4:'.,,'
  152. + First line
  153. + First line
  154. + def
  155. + First line
  156. + Second line
  157. + Third line
  158. + (7, 2)
  159. + <buffer test86.in><buffer >
  160. + baz
  161. + bar
  162. + Second line
  163. + Third line
  164. + foo
  165. *** ../vim-7.3.942/src/testdir/test87.in    2013-05-06 03:52:44.000000000 +0200
  166. --- src/testdir/test87.in    2013-05-12 21:11:43.000000000 +0200
  167. ***************
  168. *** 444,452 ****
  169.   :       endtry
  170.   :   endfor
  171.   :   call RecVars(oname)
  172. - endtry
  173.   :endfor
  174.   :only
  175.   :endfun
  176.   :"
  177.   :call Test()
  178. --- 444,493 ----
  179.   :       endtry
  180.   :   endfor
  181.   :   call RecVars(oname)
  182.   :endfor
  183.   :only
  184. + :"
  185. + :" Test buffer object
  186. + :vnew
  187. + :put ='First line'
  188. + :put ='Second line'
  189. + :put ='Third line'
  190. + :1 delete _
  191. + :py3 b=vim.current.buffer
  192. + :wincmd w
  193. + :mark a
  194. + py3 << EOF
  195. + cb = vim.current.buffer
  196. + # Tests BufferAppend and BufferItem
  197. + cb.append(b[0])
  198. + # Tests BufferSlice and BufferAssSlice
  199. + cb.append('abc') # Will be overwritten
  200. + cb[-1:] = b[:-2]
  201. + # Test BufferLength and BufferAssSlice
  202. + cb.append('def') # Will not be overwritten
  203. + cb[len(cb):] = b[:]
  204. + # Test BufferAssItem and BufferMark
  205. + cb.append('ghi') # Will be overwritten
  206. + cb[-1] = repr((len(cb) - cb.mark('a')[0], cb.mark('a')[1]))
  207. + # Test BufferRepr
  208. + cb.append(repr(cb) + repr(b))
  209. + # Modify foreign buffer
  210. + b.append('foo')
  211. + b[0]='bar'
  212. + b[0:0]=['baz']
  213. + vim.command('call append("$", getbufline(%i, 1, "$"))' % b.number)
  214. + # Test CheckBuffer
  215. + vim.command('bwipeout! ' + str(b.number))
  216. + for expr in ('b[1]','b[:] = ["A", "B"]','b[:]','b.append("abc")'):
  217. +     try:
  218. +         exec(expr)
  219. +     except vim.error:
  220. +         pass
  221. +     else:
  222. +         # Usually a SEGV here
  223. +         # Should not happen in any case
  224. +         cb.append('No exception for ' + expr)
  225. + EOF
  226.   :endfun
  227.   :"
  228.   :call Test()
  229. *** ../vim-7.3.942/src/testdir/test87.ok    2013-05-06 03:52:44.000000000 +0200
  230. --- src/testdir/test87.ok    2013-05-12 21:11:43.000000000 +0200
  231. ***************
  232. *** 295,297 ****
  233. --- 295,310 ----
  234.     G: '.,,'
  235.     W: 1:',,' 2:'.,,' 3:'.,,' 4:'.,,'
  236.     B: 1:',,' 2:'.,,' 3:'.,,' 4:'.,,'
  237. + First line
  238. + First line
  239. + def
  240. + First line
  241. + Second line
  242. + Third line
  243. + (7, 2)
  244. + <buffer test87.in><buffer >
  245. + baz
  246. + bar
  247. + Second line
  248. + Third line
  249. + foo
  250. *** ../vim-7.3.942/src/version.c    2013-05-12 20:36:09.000000000 +0200
  251. --- src/version.c    2013-05-12 21:11:53.000000000 +0200
  252. ***************
  253. *** 730,731 ****
  254. --- 730,733 ----
  255.   {   /* Add new patch number below this line */
  256. + /**/
  257. +     943,
  258.   /**/
  259.  
  260. -- 
  261. Q: Is selling software the same as selling hardware?
  262. A: No, good hardware is sold new, good software has already been used by many.
  263.  
  264.  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  265. ///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
  266. \\\  an exciting new programming language -- http://www.Zimbu.org        ///
  267.  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///
  268.