[index] Index move constants #42
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
hendrikmuhs
added a commit
that referenced
this pull request
Apr 13, 2021
…..15e4f77f 15e4f77f Corrected test names in .gitignore 93edd949 Fix typo in destination of the CMake config file 7b445fa7 Removed whitespaces at end of lines e886c605 Add support to build shared libraries on Window ca2fcd1a Merge branch 'patch-1' of https://gitlab.com/traversaro/tiny-process-library f9795d1c Increased minimum cmake version to 3.1 as suggested in #47 14a9f2e7 Add support to consume the library with find_package(tiny-process-library REQUIRED) 2e218295 Fixes #47: CMake minimal version required set to 3.0 85831b7c Add prefix to common test names d1356e66 Fixed comment 38e060e7 process_unix: Add new signal(signum) method 1b293821 No longer forces BUILD_TESTING to be ON ec5a78c1 Applied style format c5b0028f Fixed compilation on Windows a28099ab Minor cleanup of Config struct 7fea58d4 Merge branch 'win-show-window' into 'master' bf4e87ad Windows: expose the CreateProcess lpStartupInfo.wShowWindow choices 662be25a Added missing newline at end of file 40f5405c Cleanup: replaced NULL with nullptr 578466e4 Cleanup of get_exit_status and try_get_exit_status cd4f4fb6 Merge branch 'win-repeat-get-exit-status' of https://gitlab.com/ryanjmulder/tiny-process-library c9c8bf81 Do not run add_compile_options or add_definitions when tiny-process-library is a sub-project c5f8fbb7 add exit_status to windows 155706fb fix calling get_exit_status mutilple times on windows b7a49cf1 add tests for repeated call to get_exit_status df7c86bd Fixes #43: MSVC shadow warning e352957a Fixes #42: corrected include in process_win.cpp 953b19c4 No longer resolves symbolic links when Process(const std::string &command, ...) is used 93e0ed51 Cleanup of STREQUAL expression f7e76220 Update license year 3a82259c Squash -Wshorten compiler warning a6773276 Windows: CREATE_NO_WINDOW no longer set when stdout or stderr is redirected to parent process 05d9756e Unix, optimization: limit number of close calls on systems with high _SC_OPEN_MAX value bf455272 Config::inherit_file_descriptors is now supported on Windows 40a64207 Reverted last commit, and disabled config.inherit_file_descriptors on Windows due to test error in juCi++ 4091acdf Removed CREATE_NO_WINDOW flag in Windows's CreateProcess due to jucipp test fail. Will investigate further. ab8e4121 Removed windows specific settings, CREATE_NO_WINDOW is now always passed to CreateProcess. 7172fede Fixes #32 : added option to prevent inheriting file descriptors from parent process. ea3ede87 Default value for Config::windows::no_window is now true for consistent behavior across OSs 225a05f2 Fixes #35 : added Config struct for Process-constructors with no_window setting for Windows. Breaking change: constructor parameters buffer_size moved to Config struct. 6e52608b Merge branch 'poll' of https://gitlab.com/eidheim/tiny-process-library f14a022a fcntl returns 0 on success when the F_SETFL operation is used c3a25d9e Fixed tests on MacOS: do a last read on a file descriptor when it is closed 6293cf7a Unix: make read() non-blocking 5c8a3fbe Unix: simplify read loop condition, mark FDs as closed 15207735 Unix: further cleanup of async_read. Added errno == EINTR check to the poll-loop, and no longer stopps after just one of the stdout/stderr-fds are closed. Based on comments in !32 a90ea0cf Regarding #34: fixes compilation error for UNICODE as suggested by Aleksandr 38f39443 Unix: cleanup of async_read. Also no longer breaks/returns on EINTR errors. af5dd2de Unix: changed Data::exit_status to type int since the get_exit_status-functions are not thread safe, and minor improvements to the get_exit_status-functions 7d2777d3 process_unix: keep exit status (as on windows) aac992b1 process_unix: improve waitpid() error handling 13b74d3f process_unix: correctly handle waitpid() errors e3e4744e Unix: made use of poll(), running stdout and stderr readers in one thread instead of two. 273270d0 Comment cleanup 67ff6dca Windows: fixed CreateProcess call when UNICODE is used 0201606b Unix-like systems: added exits to Process::open functions b19bf49c Corrected path tests. Sometimes /bin is linked to for instance /usr/bin df1f1d9c Added arguments example a856ecc5 Added working path tests 3f103bba Added possibility to use arguments directly when creating processes b4f48df9 Fixes #31: UNICODE compilation error c6909dd9 Simplified examples link 54b95a21 Fix compilation error on Windows 92bb05ff Style format using custom clang-format a225f8cf Cleanup of comments fa54129d Improvements to environment implementation, and shorter sleeps in examples b88faa76 Added passing of environment variables for windows and linux. fae75dcd Removed build status from README.md. Using GitLab badge instead. 605fca5b Updated juCi++ link 614c0517 Migrated to GitLab 79d7ae48 Updated license year e384e4cd Added std::move to constructor member initializer lists 92ffe6b7 Fixes #29: CloseHandle no longer called on invalid handles a0348126 Updated license year 72790542 Fixed MSVC definition 88536b95 try_get_exit_status now tested on Windows git-subtree-dir: keyvi/3rdparty/tiny-process-library git-subtree-split: 15e4f77f8254e4b093f6be128db50fe4b6bee120
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
move constants file 1 directory up