aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * | viml: re-add sha256() functionNicolas Hillegeer2014-07-21
| | | | | | | | | | | | Was removed in #699 but actually doesn't have anything to do with security.
* | | build: Compile static luajit lib with -fPIC #996ZyX2014-07-25
| | | | | | | | | | | | | | | | | | | | | Should fix the following travis error: /usr/bin/ld: /opt/neovim-deps/lib/libluajit-5.1.a(lj_err.o): relocation R_X86_64_32S against `.rodata' can not be used when making a shared object; recompile with -fPIC /opt/neovim-deps/lib/libluajit-5.1.a: could not read symbols: Bad value
* | | message: add missing stdarg.h include #993Andrew Chin2014-07-25
| | | | | | | | | | | | | | | | | | Fixes up gcc 4.1 (not specifically a supported compiler but it's standard for varargs anyway so it's good to have it included and depend less on implicit includes).
* | | Merge pull request #986 from aktau/func-attr-enable-more-clangJustin M. Keyes2014-07-24
|\ \ \ | | | | | | | | func_attr: clang/icc support NONNULL_* attr
| * | | func_attr: clang/icc support NONNULL_* attrNicolas Hillegeer2014-07-23
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | Both clang and (apparently) icc support the nonnull attribute. So I'm not sure why it wasn't enabled in the first place. - clang: http://clang-analyzer.llvm.org/annotations.html#attr_nonnull - icc: https://software.intel.com/en-us/forums/topic/280290 (not conclusive evidence, but the best I could find, lacking docs)
* | | Merge pull request #987 from oni-link/fix_some_memory_leaksJustin M. Keyes2014-07-24
|\ \ \ | |/ / |/| | Fix some memory leaks in job.c and provider.c
| * | Fix memory leaks for jobs.oni-link2014-07-23
| | | | | | | | | | | | | | | Free the data memory of process and pipe handles in the close callback for a job.
| * | Fix memory leak in provider_call.oni-link2014-07-23
|/ / | | | | | | | | If no channel can be found for a method, free argument arg before returning.
* | Merge pull request #904 from war1025/dev/buffer_testsJustin M. Keyes2014-07-22
|\ \ | |/ |/| Add unit tests for buffer.c and fileio.c
| * Add tests for buffer.c and fileio.cWayne Rowcliffe2014-07-22
| |
| * Change buffer.h #defines to enums so they can be used in testsWayne Rowcliffe2014-07-22
| |
| * Statically allocate NameBuffWayne Rowcliffe2014-07-22
|/
* Merge #965 'Make os_get_localtime() portable + fix bug on win'Justin M. Keyes2014-07-20
|\
| * os/time: fix os_localtime_r on win + add func_attrNicolas Hillegeer2014-07-20
| | | | | | | | | | | | - it makes no sense for these functions to take NULL pointers - if `localtime()` on Windows returns a NULL pointer, the old code would try to dereference it.
| * os/time: make os_get_localtime more portableNicolas Hillegeer2014-07-20
| | | | | | | | | | | | gettimeofday() doesn't exist on Windows, as reported by @equalsraf. It seems a call to time() would be sufficient here, as only the seconds since the UNIX epoch are needed.
| * os/time: include time.h in os/time.hNicolas Hillegeer2014-07-20
|/ | | | Include what you use, also in the positive direction.
* Merge #962 'Implement startuptime functions on top of profiling functions'Justin M. Keyes2014-07-20
|\
| * startuptime: print vim -> nvimNicolas Hillegeer2014-07-20
| |
| * startuptime: always enable startuptimeNicolas Hillegeer2014-07-20
| | | | | | | | Removes the STARTUPTIME define.
| * globals.h: decouple vim.h and globals.hNicolas Hillegeer2014-07-20
| | | | | | | | | | | | | | | | | | Allow globals.h to be included without including vim.h. Another small piece of the puzzle of dismantling vim.h. Moving some extra `#define`'s to globals.h is no better than having them in vim.h. We should, in a later PR, move them to the file where they belong or to a separate `constants.h` or something.
| * garray: add necessary includesNicolas Hillegeer2014-07-20
| | | | | | | | The upcoming decoupling of vim.h and globals.h needs this to compile.
| * startuptime: implement on top of profiling fnsNicolas Hillegeer2014-07-20
| | | | | | | | | | | | Reuse the profiling functions to implement the startuptime functions. Decreases our dependency on `gettimeofday()` and thus gets us a little bit closer to a clean port to Windows.
| * startuptime: move code to profile.{c,h} + docNicolas Hillegeer2014-07-20
|/ | | | | It's a better place to put it. Also slightly documented and reformatted, but not changed.
* coverity/62610: Remove dead code from setfname()Wesley Wiser2014-07-20
|
* types: typedef long_u as uintptr_t #961Nicolas Hillegeer2014-07-20
| | | | As discussed here: https://github.com/neovim/neovim/pull/941/files#r15026398
* Merge PR #895 'Core service providers...'Thiago de Arruda2014-07-17
|\
| * api tests: set 'initpython' in BeforeEachTestThiago de Arruda2014-07-17
| | | | | | | | This is required to run some tests of the python client
| * events: Refactor how event deferral is handledThiago de Arruda2014-07-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Remove all *_set_defer methods and the 'defer' flag from rstream/jobs - Added {signal,rstream,job}_event_source functions. Each return a pointer that represent the event source for the object in question(For signals, a static pointer is returned) - Added a 'source' field to the Event struct, which is set to the appropriate value by the code that created the event. - Added a 'sources' parameter to `event_poll`. It should point to a NULL-terminated array of event sources that will be used to decide which events should be processed immediately - Added a 'source_override' parameter to `rstream_new`. This was required to use jobs as event sources of RStream instances(When "focusing" on a job, for example). - Extracted `process_from` static function from `event_process`. - Remove 'defer' parameter from `event_process`, which now operates only on deferred events. - Refactor `channel_send_call` to use the new lock mechanism What changed in a single sentence: Code that calls `event_poll` have to specify which event sources should NOT be deferred. This change was necessary for a number of reasons: - To fix a bug where due to race conditions, a client request could end in the deferred queue in the middle of a `channel_send_call` invocation, resulting in a deadlock since the client process would never receive a response, and channel_send_call would never return because the client would still be waiting for the response. - To handle "event locking" correctly in recursive `channel_send_call` invocations when the frames are waiting for responses from different clients. Not much of an issue now since there's only a python client, but could break things later. - To simplify the process of implementing synchronous functions that depend on asynchronous events.
| * api/events/msgpack: Insert log statements to improve debuggingThiago de Arruda2014-07-17
| | | | | | | | | | | | Also changed the default log level to INFO so developers won't end up with big log files without asking explicitly(DLOG statements were placed in really "hot" code)
| * getchar: fix infinite loop due to pending eventsThiago de Arruda2014-07-17
| | | | | | | | | | The `inchar` function could enter an infinite loop if there are events pending to be processed when an interrupt is received.
| * api: make buffer_{get,set}_slice automatically assume `include_end`Thiago de Arruda2014-07-17
| | | | | | | | | | This is for compatibility with python-vim interface: When passing an end index with a value higher than the last index, assume the `include_end` flag
| * provider: Add support for clipboard registers.Thiago de Arruda2014-07-17
| | | | | | | | | | | | | | | | | | | | | | | | | | This reimplements the '+'/'*' clipboard registers(both are aliases to the same register, no dedicated storage for the X11 selection) on top of the provider infrastructure. This adds two new 'unnamedclip' option, has the same effect of setting 'clipboard' to 'unnamed/unnamedplus' in vim The 'clipboard' option was not reused because all values(except 'unnamedplus') seem to be useless for Neovim, and the code to parse the option was relatively big. The option remains for vim compatibility but it's silently ignored.
| * provider: Add support for python commands/functionsThiago de Arruda2014-07-17
| | | | | | | | | | | | | | | | This uses the provider/scripting infrastructure to reintroduce python support through the msgpack-rpc API. A new 'initpython' option was added, and it must be set to a command that will bootstrap the python provider the first time it's needed.
| * provider: Add support functions for calling external interpretersThiago de Arruda2014-07-17
| | | | | | | | | | | | This uses the provider module infrastructure to implement common code for vimscript commands/functions that need to communicate with external interpreters, eg: pydo, rubydo, pyfile, rubyfile, etc.
| * provider: New module used to expose extension points for core servicesThiago de Arruda2014-07-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Introducing the concept of providers: co-processes that talk with the editor through the remote API and provide implementation for one or more core services. The `provider_register` function and it's API wrapper can be used by channels that want to self-register as a service provider. Some old builtin vim features will be re-implemented as providers. The `provider_has_feature` function is used to check if a provider implementing a certain feature is available(It will be called by the `has` vimscript function to check for features in a vim-compatible way) This implements the provider module without exposing any extension points, which will be done in future commits.
| * channel: Bugfixes and refactorThiago de Arruda2014-07-17
| | | | | | | | | | | | | | | | | | | | - All functions that require a channel id will fail when the channel was disabled - Rewrite `call_stack_unwind` as `call_set_error`. It will now disable the channel and set error on all frames. The stack will be unwinded automatically while the involved functions exit. - Remove `disable_channel` function. If channels are disabled, they will be closed as soon as possible
| * job: No longer free the job data. It's now done by the exit callbackThiago de Arruda2014-07-17
| |
| * channel: Implement channel_exists functionThiago de Arruda2014-07-17
| |
| * channel: Refactor channel_from_job to return the channel idThiago de Arruda2014-07-17
| |
| * api: Refactor write_msg to use separate out/err buffersThiago de Arruda2014-07-17
| |
| * wstream: Pass WBuffer refcount as a constructor parameterThiago de Arruda2014-07-17
| | | | | | | | | | | | This is required to handle broadcasting when the first write fails. Ref: https://github.com/tarruda/neovim/commit/11916b6b595421ce2ece10f7aa40757cc4937c0c#commitcomment-6792287
| * wstream: document default value for 'maxmem'Thiago de Arruda2014-07-17
|/
* Merge pull request #964 '[RFC] Fix #963'Thiago de Arruda2014-07-17
|\
| * Fix #963.Pavel Platto2014-07-17
|/ | | | | | Problem: Bug was introduced because `os_open` returns `-errno` in case of an error instead of just `-1` which was returned by `mch_open`. Solution: Check return value with `< 0` instead of `== -1`.
* Merge #928 'Move po gen to cmake'John Szakmeister2014-07-17
|\ | | | | | | This includes the necessary fixes to sv.po to make the build work again.
| * Fix some newline issues in the sv translation.John Szakmeister2014-07-17
| |
| * Move po generation from Make to CMake.John Szakmeister2014-07-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes #902: localization build: `install`, `uninstall`, `prefixcheck targets. All the language-related bits will now build under CMake. Changes include: * Moving all non-generated sources into the NEOVIM_SOURCES variable to aid in generating the .pot file. * Moving a couple generated sources from NEOVIM_SOURCES and into NEOVIM_GENERATED_SOURCES. * Added NEOVIM_HEADERS to the executable and the library for folks who are using something other than Ninja or makefiles (that way the headers will show up in the IDE files). * Now uses gettext's `--update` switch to update the .po files, rather than doing a fragile `mv` dance that could leave you with a broken working tree if you press CTRL-C at the right time. * Creates `update-po-${LANG}` targets for updating individual languages, just like the original Makefile. * Also adds the `update-po` target for updating all the languages. * Ported the `check-${LANG}` style targets. They're `check-po-${LANG}` under CMake. * Handles all the one-off instances that were in the original Makefile. Fixed up ko.UTF-8.po to include the "Original translation" line like other .po files to make the generation of the "Generate from ..." comments consistent. Updated ko.po with the new text.
| * Customize the log file for the check targets based on the input file.John Szakmeister2014-07-17
|/ | | | | | The idea being that it's better to segregate feedback, just in case someone is working on several translations. Now the check log will appear in `./build/src/nvim/po/check-${LANG}.log`.
* Revert "Merge #928 'Move po gen to cmake'"John Szakmeister2014-07-17
| | | | | This reverts commit 53fba180c32724809d8ab35238eea68440b7ac97, reversing changes made to a3b3db8256c259d10957d45666cf4179e328f3f4.
* Merge #928 'Move po gen to cmake'John Szakmeister2014-07-17
|\