Completed
Pull Request — master (#179)
by Björn
02:26
created

neovim.api.Nvim.run_loop()   A

Complexity

Conditions 3

Size

Total Lines 10

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 6
CRAP Score 3.0261
Metric Value
cc 3
dl 0
loc 10
ccs 6
cts 7
cp 0.8571
crap 3.0261
rs 9.4285

2 Methods

Rating   Name   Duplication   Size   Complexity  
A neovim.api.Nvim.filter_notification_cb() 0 2 1
A neovim.api.Nvim.filter_request_cb() 0 3 1
1
"""Main Nvim interface."""
2 6
import functools
3 6
import os
4
5 6
from traceback import format_exc, format_stack
6
7 6
from msgpack import ExtType
0 ignored issues
show
Configuration introduced by
The import msgpack could not be resolved.

This can be caused by one of the following:

1. Missing Dependencies

This error could indicate a configuration issue of Pylint. Make sure that your libraries are available by adding the necessary commands.

# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3
Tip: We are currently not using virtualenv to run pylint, when installing your modules make sure to use the command for the correct version.

2. Missing __init__.py files

This error could also result from missing __init__.py files in your module folders. Make sure that you place one file in each sub-folder.

Loading history...
8
9 6
from .buffer import Buffer
10 6
from .common import (DecodeHook, Remote, RemoteApi, RemoteMap, RemoteSequence, walk, ApiMethod)
11 6
from .tabpage import Tabpage
12 6
from .window import Window
13 6
from ..compat import IS_PYTHON3
14
15
16 6
__all__ = ('Nvim')
17
18
19 6
os_chdir = os.chdir
20
21
22 6
class Nvim(object):
23
24
    """Class that represents a remote Nvim instance.
25
26
    This class is main entry point to Nvim remote API, it is a thin wrapper
27
    around Session instances.
28
29
    The constructor of this class must not be called directly. Instead, the
30
    `from_session` class method should be used to create the first instance
31
    from a raw `Session` instance.
32
33
    Subsequent instances for the same session can be created by calling the
34
    `with_hook` instance method and passing a SessionHook instance. This can
35
    be useful to have multiple `Nvim` objects that behave differently without
36
    one affecting the other.
37
    """
38
39 6
    @classmethod
40
    def from_session(cls, session):
41
        """Create a new Nvim instance for a Session instance.
42
43
        This method must be called to create the first Nvim instance, since it
44
        queries Nvim metadata for type information and sets a SessionHook for
45
        creating specialized objects from Nvim remote handles.
46
        """
47 6
        session.error_wrapper = lambda e: NvimError(e[1])
48 6
        channel_id, metadata = session.request(b'vim_get_api_info')
49
50 6
        if IS_PYTHON3:
51
            # decode all metadata strings for python3
52 5
            metadata = DecodeHook().walk(metadata)
53
54 6
        types = {
55
            metadata['types']['Buffer']['id']: Buffer,
56
            metadata['types']['Window']['id']: Window,
57
            metadata['types']['Tabpage']['id']: Tabpage,
58
        }
59
60 6
        return cls(session, channel_id, metadata, types)
61
62 6
    @classmethod
63
    def from_nvim(cls, nvim):
64
        return cls(nvim.session, nvim.channel_id, nvim.metadata, nvim.types, nvim._decodehook)
0 ignored issues
show
Coding Style Best Practice introduced by
It seems like _decodehook was declared protected and should not be accessed from this context.

Prefixing a member variable _ is usually regarded as the equivalent of declaring it with protected visibility that exists in other languages. Consequentially, such a member should only be accessed from the same class or a child class:

class MyParent:
    def __init__(self):
        self._x = 1;
        self.y = 2;

class MyChild(MyParent):
    def some_method(self):
        return self._x    # Ok, since accessed from a child class

class AnotherClass:
    def some_method(self, instance_of_my_child):
        return instance_of_my_child._x   # Would be flagged as AnotherClass is not
                                         # a child class of MyParent
Loading history...
65
66 6
    def __init__(self, session, channel_id, metadata, types, decodehook=None):
67
        """Initialize a new Nvim instance. This method is module-private."""
68 6
        self._session = session
69 6
        self.channel_id = channel_id
70 6
        self.metadata = metadata
71 6
        self.types = types
72 6
        self.api = RemoteApi(session, 'vim_')
73 6
        self.vars = RemoteMap(self, 'vim_get_var', 'vim_set_var')
74 6
        self.vvars = RemoteMap(self, 'vim_get_vvar', None)
75 6
        self.options = RemoteMap(self, 'vim_get_option', 'vim_set_option')
76 6
        self.buffers = RemoteSequence(self, 'vim_get_buffers')
77 6
        self.windows = RemoteSequence(self, 'vim_get_windows')
78 6
        self.tabpages = RemoteSequence(self, 'vim_get_tabpages')
79 6
        self.current = Current(self)
80 6
        self.funcs = Funcs(self)
81 6
        self.error = NvimError
82 6
        self._decodehook = decodehook
83
84 6
    def _from_nvim(self, obj):
85 6
        if type(obj) is ExtType:
86 6
            cls = self.types[obj.code]
87 6
            return cls(self, (obj.code, obj.data))
88 6
        if self._decodehook is not None:
89 3
            obj = self._decodehook.decode_if_bytes(obj)
90 6
        return obj
91
92
    # TODO: it would be beterr if objects seralize themselves
93 6
    def _to_nvim(self, obj):
0 ignored issues
show
Coding Style introduced by
This method could be written as a function/class method.

If a method does not access any attributes of the class, it could also be implemented as a function or static method. This can help improve readability. For example

class Foo:
    def some_method(self, x, y):
        return x + y;

could be written as

class Foo:
    @classmethod
    def some_method(cls, x, y):
        return x + y;
Loading history...
94 6
        if isinstance(obj, Remote):
95 6
            return ExtType(*obj.code_data)
96 6
        return obj
97
98 6
    def request(self, name, *args, **kwargs):
99
        """Wrapper for Session.request."""
100 6
        args = walk(self._to_nvim, args)
101 6
        return walk(self._from_nvim, self._session.request(name, *args, **kwargs))
102
103 6
    def next_message(self):
104
        """Wrapper for Session.next_message."""
105 6
        msg = self._session.next_message()
106 6
        if msg:
107 6
            return walk(self._from_nvim, msg)
108
109 6
    def run_loop(self, request_cb, notification_cb, setup_cb=None):
110
        """Wrapper for Session.run."""
111 6
        def filter_request_cb(name, args):
112 6
            result = request_cb(self._from_nvim(name), walk(self._from_nvim, args))
113 6
            return walk(self._to_nvim, result)
114
115 6
        def filter_notification_cb(name, args):
116
            notification_cb(self._from_nvim(name), walk(self._from_nvim, args))
117
118 6
        self._session.run(filter_request_cb, filter_notification_cb, setup_cb)
119
120 6
    def stop_loop(self):
121
        """Wrapper for Session.stop."""
122 6
        self._session.stop()
123
124 6
    def with_decodehook(self, hook):
125
        """Initialize a new Nvim instance."""
126 3
        return Nvim(self.session, self.channel_id,
127
                    self.metadata, self.types, hook)
128
129 6
    @property
130
    def session(self):
131
        """Return the Session or SessionFilter for a Nvim instance."""
132 6
        return self._session
133
134 6
    def ui_attach(self, width, height, rgb):
135
        """Register as a remote UI.
136
137
        After this method is called, the client will receive redraw
138
        notifications.
139
        """
140 6
        return self.request('ui_attach', width, height, rgb)
141
142 6
    def ui_detach(self):
143
        """Unregister as a remote UI."""
144 6
        return self.request('ui_detach')
145
146 6
    def ui_try_resize(self, width, height):
147
        """Notify nvim that the client window has resized.
148
149
        If possible, nvim will send a redraw request to resize.
150
        """
151
        return self.request('ui_try_resize', width, height)
152
153 6
    def subscribe(self, event):
154
        """Subscribe to a Nvim event."""
155 6
        return self.request('vim_subscribe', event)
156
157 6
    def unsubscribe(self, event):
158
        """Unsubscribe to a Nvim event."""
159 6
        return self.request('vim_unsubscribe', event)
160
161 6
    command = ApiMethod("vim_command")
162 6
    command_output = ApiMethod("vim_command_output")
163 6
    eval = ApiMethod("vim_eval")
164
165
166 6
    def call(self, name, *args, **kwargs):
167
        """Call a vimscript function."""
168 6
        for k in kwargs:
169 6
            if k != "async":
170
                raise TypeError(
171
                    "call() got an unexpected keyword argument '{}'".format(k))
172 6
        return self.request('vim_call_function', name, args, **kwargs)
173
174 6
    def strwidth(self, string):
175
        """Return the number of display cells `string` occupies.
176
177
        Tab is counted as one cell.
178
        """
179 6
        return self.request('vim_strwidth', string)
180
181 6
    def list_runtime_paths(self):
182
        """Return a list of paths contained in the 'runtimepath' option."""
183
        return self.request('vim_list_runtime_paths')
184
185 6
    def foreach_rtp(self, cb):
186
        """Invoke `cb` for each path in 'runtimepath'.
187
188
        Call the given callable for each path in 'runtimepath' until either
189
        callable returns something but None, the exception is raised or there
190
        are no longer paths. If stopped in case callable returned non-None,
191
        vim.foreach_rtp function returns the value returned by callable.
192
        """
193
        for path in self.request('vim_list_runtime_paths'):
194
            try:
195
                if cb(path) is not None:
196
                    break
197
            except Exception:
0 ignored issues
show
Best Practice introduced by
Catching very general exceptions such as Exception is usually not recommended.

Generally, you would want to handle very specific errors in the exception handler. This ensure that you do not hide other types of errors which should be fixed.

So, unless you specifically plan to handle any error, consider adding a more specific exception.

Loading history...
198
                break
199
200 6
    def chdir(self, dir_path):
201
        """Run os.chdir, then all appropriate vim stuff."""
202 6
        os_chdir(dir_path)
203 6
        return self.request('vim_change_directory', dir_path)
204
205 6
    def feedkeys(self, keys, options='', escape_csi=True):
206
        """Push `keys` to Nvim user input buffer.
207
208
        Options can be a string with the following character flags:
209
        - 'm': Remap keys. This is default.
210
        - 'n': Do not remap keys.
211
        - 't': Handle keys as if typed; otherwise they are handled as if coming
212
               from a mapping. This matters for undo, opening folds, etc.
213
        """
214
        return self.request('vim_feedkeys', keys, options, escape_csi)
215
216 6
    def input(self, bytes):
0 ignored issues
show
Bug Best Practice introduced by
This seems to re-define the built-in bytes.

It is generally discouraged to redefine built-ins as this makes code very hard to read.

Loading history...
217
        """Push `bytes` to Nvim low level input buffer.
218
219
        Unlike `feedkeys()`, this uses the lowest level input buffer and the
220
        call is not deferred. It returns the number of bytes actually
221
        written(which can be less than what was requested if the buffer is
222
        full).
223
        """
224 6
        return self.request('vim_input', bytes)
225
226 6
    def replace_termcodes(self, string, from_part=False, do_lt=True,
227
                          special=True):
228
        r"""Replace any terminal code strings by byte sequences.
229
230
        The returned sequences are Nvim's internal representation of keys,
231
        for example:
232
233
        <esc> -> '\x1b'
234
        <cr>  -> '\r'
235
        <c-l> -> '\x0c'
236
        <up>  -> '\x80ku'
237
238
        The returned sequences can be used as input to `feedkeys`.
239
        """
240 1
        return self.request('vim_replace_termcodes', string,
241
                                     from_part, do_lt, special)
242
243 6
    def out_write(self, msg):
244
        """Print `msg` as a normal message."""
245
        return self.request('vim_out_write', msg)
246
247 6
    def err_write(self, msg, async=False):
248
        """Print `msg` as an error message."""
249
        return self.request('vim_err_write', msg, async=async)
250
251 6
    def quit(self, quit_command='qa!'):
252
        """Send a quit command to Nvim.
253
254
        By default, the quit command is 'qa!' which will make Nvim quit without
255
        saving anything.
256
        """
257
        try:
258
            self.command(quit_command)
259
        except IOError:
0 ignored issues
show
Unused Code introduced by
This except handler seems to be unused and could be removed.

Except handlers which only contain pass and do not have an else clause can usually simply be removed:

try:
    raises_exception()
except:  # Could be removed
    pass
Loading history...
260
            # sending a quit command will raise an IOError because the
261
            # connection is closed before a response is received. Safe to
262
            # ignore it.
263
            pass
264
265 6
    def new_highlight_source(self):
266
        """Return new src_id for use with Buffer.add_highlight."""
267
        return self.current.buffer.add_highlight("", 0, src_id=0)
268
269 6
    def async_call(self, fn, *args, **kwargs):
270
        """Schedule `fn` to be called by the event loop soon.
271
272
        This function is thread-safe, and is the only way code not
273
        on the main thread could interact with nvim api objects.
274
275
        This function can also be called in a synchronous
276
        event handler, just before it returns, to defer execution
277
        that shouldn't block neovim.
278
        """
279 6
        call_point = ''.join(format_stack(None, 5)[:-1])
280
281 6
        def handler():
282 6
            try:
283 6
                fn(*args, **kwargs)
284
            except Exception as err:
285
                msg = ("error caught while executing async callback:\n"
286
                       "{!r}\n{}\n \nthe call was requested at\n{}"
287
                       .format(err, format_exc(5), call_point))
288
                self.err_write(msg, async=True)
289
                raise
290 6
        self._session.threadsafe_call(handler)
291
292
293 6
class Current(object):
294
295
    """Helper class for emulating vim.current from python-vim."""
296
297 6
    def __init__(self, session):
298 6
        self._session = session
299 6
        self.range = None
300
301 6
    @property
302
    def line(self):
303 6
        return self._session.request('vim_get_current_line')
304
305 6
    @line.setter
306
    def line(self, line):
307 6
        return self._session.request('vim_set_current_line', line)
308
309 6
    @property
310
    def buffer(self):
311 6
        return self._session.request('vim_get_current_buffer')
312
313 6
    @buffer.setter
314
    def buffer(self, buffer):
315 6
        return self._session.request('vim_set_current_buffer', buffer)
316
317 6
    @property
318
    def window(self):
319 6
        return self._session.request('vim_get_current_window')
320
321 6
    @window.setter
322
    def window(self, window):
323 6
        return self._session.request('vim_set_current_window', window)
324
325 6
    @property
326
    def tabpage(self):
327 6
        return self._session.request('vim_get_current_tabpage')
328
329 6
    @tabpage.setter
330
    def tabpage(self, tabpage):
331 6
        return self._session.request('vim_set_current_tabpage', tabpage)
332
333
334 6
class Funcs(object):
335
336
    """Helper class for functional vimscript interface."""
337
338 6
    def __init__(self, nvim):
339 6
        self._nvim = nvim
340
341 6
    def __getattr__(self, name):
342 6
        return functools.partial(self._nvim.call, name)
343
344
345
346 6
class NvimError(Exception):
347
    pass
348