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.808 < prev    next >
Encoding:
Internet Message Format  |  2013-02-12  |  8.0 KB
  1. To: vim_dev@googlegroups.com
  2. Subject: Patch 7.3.808
  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.808
  11. Problem:    Python threads still do not work properly.
  12. Solution:   Fix both Python 2 and 3.  Add tests. (Ken Takata)
  13. Files:        src/if_python.c, 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.807/src/if_python.c    2013-01-30 11:44:33.000000000 +0100
  19. --- src/if_python.c    2013-02-13 14:07:28.000000000 +0100
  20. ***************
  21. *** 741,747 ****
  22.       PyMac_Initialize();
  23.   #endif
  24.       /* Initialise threads, and below save the state using
  25. !      * PyGILState_Ensure.  Without the call to PyGILState_Ensure, thread
  26.        * specific state (such as the system trace hook), will be lost
  27.        * between invocations of Python code. */
  28.       PyEval_InitThreads();
  29. --- 741,747 ----
  30.       PyMac_Initialize();
  31.   #endif
  32.       /* Initialise threads, and below save the state using
  33. !      * PyEval_SaveThread.  Without the call to PyEval_SaveThread, thread
  34.        * specific state (such as the system trace hook), will be lost
  35.        * between invocations of Python code. */
  36.       PyEval_InitThreads();
  37. ***************
  38. *** 755,764 ****
  39.       if (PythonMod_Init())
  40.           goto fail;
  41.   
  42. -     /* The first python thread is vim's, release the lock. */
  43. -     Python_SaveThread();
  44. -     pygilstate = PyGILState_Ensure();
  45. - 
  46.       globals = PyModule_GetDict(PyImport_AddModule("__main__"));
  47.   
  48.       /* Remove the element from sys.path that was added because of our
  49. --- 755,760 ----
  50. ***************
  51. *** 767,773 ****
  52.        * the current directory in sys.path. */
  53.       PyRun_SimpleString("import sys; sys.path = filter(lambda x: x != '/must>not&exist', sys.path)");
  54.   
  55. !     PyGILState_Release(pygilstate);
  56.   
  57.       initialised = 1;
  58.       }
  59. --- 763,776 ----
  60.        * the current directory in sys.path. */
  61.       PyRun_SimpleString("import sys; sys.path = filter(lambda x: x != '/must>not&exist', sys.path)");
  62.   
  63. !     /* lock is created and acquired in PyEval_InitThreads() and thread
  64. !      * state is created in Py_Initialize()
  65. !      * there _PyGILState_NoteThreadState() also sets gilcounter to 1
  66. !      * (python must have threads enabled!)
  67. !      * so the following does both: unlock GIL and save thread state in TLS
  68. !      * without deleting thread state
  69. !      */
  70. !     PyEval_SaveThread();
  71.   
  72.       initialised = 1;
  73.       }
  74. *** ../vim-7.3.807/src/if_python3.c    2012-11-28 15:33:10.000000000 +0100
  75. --- src/if_python3.c    2013-02-13 14:07:28.000000000 +0100
  76. ***************
  77. *** 729,741 ****
  78.   #else
  79.       PyMac_Initialize();
  80.   #endif
  81. !     /* Initialise threads, and save the state using PyGILState_Ensure.
  82. !      * Without the call to PyGILState_Ensure, thread specific state (such
  83. !      * as the system trace hook), will be lost between invocations of
  84. !      * Python code. */
  85.       PyEval_InitThreads();
  86. -     pygilstate = PyGILState_Ensure();
  87. - 
  88.   #ifdef DYNAMIC_PYTHON3
  89.       get_py3_exceptions();
  90.   #endif
  91. --- 729,739 ----
  92.   #else
  93.       PyMac_Initialize();
  94.   #endif
  95. !     /* Initialise threads, and below save the state using
  96. !      * PyEval_SaveThread.  Without the call to PyEval_SaveThread, thread
  97. !      * specific state (such as the system trace hook), will be lost
  98. !      * between invocations of Python code. */
  99.       PyEval_InitThreads();
  100.   #ifdef DYNAMIC_PYTHON3
  101.       get_py3_exceptions();
  102.   #endif
  103. ***************
  104. *** 754,766 ****
  105.        */
  106.       PyRun_SimpleString("import vim; import sys; sys.path = list(filter(lambda x: not x.endswith('must>not&exist'), sys.path))");
  107.   
  108. !     // lock is created and acquired in PyEval_InitThreads() and thread
  109. !     // state is created in Py_Initialize()
  110. !     // there _PyGILState_NoteThreadState() also sets gilcounter to 1
  111. !     // (python must have threads enabled!)
  112. !     // so the following does both: unlock GIL and save thread state in TLS
  113. !     // without deleting thread state
  114. !     PyGILState_Release(pygilstate);
  115.   
  116.       py3initialised = 1;
  117.       }
  118. --- 752,765 ----
  119.        */
  120.       PyRun_SimpleString("import vim; import sys; sys.path = list(filter(lambda x: not x.endswith('must>not&exist'), sys.path))");
  121.   
  122. !     /* lock is created and acquired in PyEval_InitThreads() and thread
  123. !      * state is created in Py_Initialize()
  124. !      * there _PyGILState_NoteThreadState() also sets gilcounter to 1
  125. !      * (python must have threads enabled!)
  126. !      * so the following does both: unlock GIL and save thread state in TLS
  127. !      * without deleting thread state
  128. !      */
  129. !     PyEval_SaveThread();
  130.   
  131.       py3initialised = 1;
  132.       }
  133. *** ../vim-7.3.807/src/testdir/test86.in    2012-09-21 14:00:05.000000000 +0200
  134. --- src/testdir/test86.in    2013-02-13 13:58:25.000000000 +0100
  135. ***************
  136. *** 267,272 ****
  137. --- 267,320 ----
  138.   :      $put =toput
  139.   :   endtry
  140.   :endfor
  141. + :"
  142. + :" threading
  143. + :let l = [0]
  144. + :py l=vim.bindeval('l')
  145. + :py <<EOF
  146. + import threading
  147. + import time
  148. + 
  149. + class T(threading.Thread):
  150. +     def __init__(self):
  151. +         threading.Thread.__init__(self)
  152. +         self.t = 0
  153. +         self.running = True
  154. + 
  155. +     def run(self):
  156. +         while self.running:
  157. +             self.t += 1
  158. +             time.sleep(0.1)
  159. + 
  160. + t = T()
  161. + t.start()
  162. + EOF
  163. + :sleep 1
  164. + :py t.running = False
  165. + :py t.join()
  166. + :py l[0] = t.t > 8  # check if the background thread is working
  167. + :$put =string(l)
  168. + :"
  169. + :" settrace
  170. + :let l = []
  171. + :py l=vim.bindeval('l')
  172. + :py <<EOF
  173. + import sys
  174. + 
  175. + def traceit(frame, event, arg):
  176. +     global l
  177. +     if event == "line":
  178. +         l.extend([frame.f_lineno])
  179. +     return traceit
  180. + 
  181. + def trace_main():
  182. +     for i in range(5):
  183. +         pass
  184. + EOF
  185. + :py sys.settrace(traceit)
  186. + :py trace_main()
  187. + :py sys.settrace(None)
  188. + :$put =string(l)
  189.   :endfun
  190.   :"
  191.   :call Test()
  192. *** ../vim-7.3.807/src/testdir/test86.ok    2012-09-21 14:00:05.000000000 +0200
  193. --- src/testdir/test86.ok    2013-02-13 13:58:25.000000000 +0100
  194. ***************
  195. *** 63,65 ****
  196. --- 63,67 ----
  197.   {"\0": 1}:    Vim(let):E859:
  198.   undefined_name:    Vim(let):E858:
  199.   vim:    Vim(let):E859:
  200. + [1]
  201. + [1, 10, 11, 10, 11, 10, 11, 10, 11, 10, 11, 10, 1]
  202. *** ../vim-7.3.807/src/testdir/test87.in    2012-09-21 14:00:05.000000000 +0200
  203. --- src/testdir/test87.in    2013-02-13 13:58:25.000000000 +0100
  204. ***************
  205. *** 267,272 ****
  206. --- 267,320 ----
  207.   :      $put =toput
  208.   :   endtry
  209.   :endfor
  210. + :"
  211. + :" threading
  212. + :let l = [0]
  213. + :py3 l=vim.bindeval('l')
  214. + :py3 <<EOF
  215. + import threading
  216. + import time
  217. + 
  218. + class T(threading.Thread):
  219. +     def __init__(self):
  220. +         threading.Thread.__init__(self)
  221. +         self.t = 0
  222. +         self.running = True
  223. + 
  224. +     def run(self):
  225. +         while self.running:
  226. +             self.t += 1
  227. +             time.sleep(0.1)
  228. + 
  229. + t = T()
  230. + t.start()
  231. + EOF
  232. + :sleep 1
  233. + :py3 t.running = False
  234. + :py3 t.join()
  235. + :py3 l[0] = t.t > 8  # check if the background thread is working
  236. + :$put =string(l)
  237. + :"
  238. + :" settrace
  239. + :let l = []
  240. + :py3 l=vim.bindeval('l')
  241. + :py3 <<EOF
  242. + import sys
  243. + 
  244. + def traceit(frame, event, arg):
  245. +     global l
  246. +     if event == "line":
  247. +         l += [frame.f_lineno]
  248. +     return traceit
  249. + 
  250. + def trace_main():
  251. +     for i in range(5):
  252. +         pass
  253. + EOF
  254. + :py3 sys.settrace(traceit)
  255. + :py3 trace_main()
  256. + :py3 sys.settrace(None)
  257. + :$put =string(l)
  258.   :endfun
  259.   :"
  260.   :call Test()
  261. *** ../vim-7.3.807/src/testdir/test87.ok    2012-09-21 14:00:05.000000000 +0200
  262. --- src/testdir/test87.ok    2013-02-13 13:58:25.000000000 +0100
  263. ***************
  264. *** 63,65 ****
  265. --- 63,67 ----
  266.   {"\0": 1}:    Vim(let):E861:
  267.   undefined_name:    Vim(let):E860:
  268.   vim:    Vim(let):E861:
  269. + [1]
  270. + [1, 10, 11, 10, 11, 10, 11, 10, 11, 10, 11, 10, 1]
  271. *** ../vim-7.3.807/src/version.c    2013-02-13 12:15:59.000000000 +0100
  272. --- src/version.c    2013-02-13 14:10:53.000000000 +0100
  273. ***************
  274. *** 727,728 ****
  275. --- 727,730 ----
  276.   {   /* Add new patch number below this line */
  277. + /**/
  278. +     808,
  279.   /**/
  280.  
  281. -- 
  282. Lawmakers made it obligatory for everybody to take at least one bath
  283. each week -- on Saturday night.
  284.         [real standing law in Vermont, United States of America]
  285.  
  286.  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  287. ///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
  288. \\\  an exciting new programming language -- http://www.Zimbu.org        ///
  289.  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///
  290.