summaryrefslogtreecommitdiffstats
path: root/lldb/tools/lldb-mi/MICmnStreamStdin.cpp
Commit message (Collapse)AuthorAgeFilesLines
* [LLDB] Remove lldb-miJonas Devlieghere2019-07-181-218/+0
| | | | | | | | | | | | | | As discussed on the mailing list [1], this patch removes the lldb-mi tool and its tests from the LLDB repository. We moved lldb-mi into a separate repository on GitHub [2] for downstream users or maintainers to build and package. [1] http://lists.llvm.org/pipermail/lldb-dev/2019-July/015103.html [2] https://github.com/lldb-tools/lldb-mi Differential revision: https://reviews.llvm.org/D64255 llvm-svn: 366465
* [NFC] Remove ASCII lines from commentsJonas Devlieghere2019-04-101-9/+0
| | | | | | | | | | | | | | | | | | | | | | | A lot of comments in LLDB are surrounded by an ASCII line to delimit the begging and end of the comment. Its use is not really consistent across the code base, sometimes the lines are longer, sometimes they are shorter and sometimes they are omitted. Furthermore, it looks kind of weird with the 80 column limit, where the comment actually extends past the line, but not by much. Furthermore, when /// is used for Doxygen comments, it looks particularly odd. And when // is used, it incorrectly gives the impression that it's actually a Doxygen comment. I assume these lines were added to improve distinguishing between comments and code. However, given that todays editors and IDEs do a great job at highlighting comments, I think it's worth to drop this for the sake of consistency. The alternative is fixing all the inconsistencies, which would create a lot more churn. Differential revision: https://reviews.llvm.org/D60508 llvm-svn: 358135
* Update the file headers across all of the LLVM projects in the monorepoChandler Carruth2019-01-191-4/+3
| | | | | | | | | | | | | | | | | to reflect the new license. We understand that people may be surprised that we're moving the header entirely to discuss the new license. We checked this carefully with the Foundation's lawyer and we believe this is the correct approach. Essentially, all code in the project is now made available by the LLVM project under our new license, so you will see that the license headers include that license only. Some of our contributors have contributed code under our old license, and accordingly, we have retained a copy of our old license notice in the top-level files in each project and repository. llvm-svn: 351636
* Remove comments after header includes.Jonas Devlieghere2018-11-111-1/+1
| | | | | | | | | | This patch removes the comments following the header includes. They were added after running IWYU over the LLDB codebase. However they add little value, are often outdates and burdensome to maintain. Differential revision: https://reviews.llvm.org/D54385 llvm-svn: 346625
* Fix build for mingw.Hafiz Abid Qadeer2016-12-151-1/+1
| | | | | | | | | | | | Summary: I was building lldb using cross mingw-w64 toolchain on Linux and observed some issues. This is first patch in the series to fix that build. It mostly corrects the case of include files and adjusts some #ifdefs from _MSC_VER to _WIN32 and vice versa. I built lldb on windows with VS after applying this patch to make sure it does not break the build there. Reviewers: zturner, labath, abidh Subscribers: ki.stfu, mgorny, lldb-commits Differential Revision: https://reviews.llvm.org/D27759 llvm-svn: 289821
* *** This commit represents a complete reformatting of the LLDB source codeKate Stone2016-09-061-130/+117
| | | | | | | | | | | | | | | | | | | | | | | *** to conform to clang-format’s LLVM style. This kind of mass change has *** two obvious implications: Firstly, merging this particular commit into a downstream fork may be a huge effort. Alternatively, it may be worth merging all changes up to this commit, performing the same reformatting operation locally, and then discarding the merge for this particular commit. The commands used to accomplish this reformatting were as follows (with current working directory as the root of the repository): find . \( -iname "*.c" -or -iname "*.cpp" -or -iname "*.h" -or -iname "*.mm" \) -exec clang-format -i {} + find . -iname "*.py" -exec autopep8 --in-place --aggressive --aggressive {} + ; The version of clang-format used was 3.9.0, and autopep8 was 1.2.4. Secondly, “blame” style tools will generally point to this commit instead of a meaningful prior commit. There are alternatives available that will attempt to look through this change and find the appropriate prior commit. YMMV. llvm-svn: 280751
* [lldb-mi] Use empty arg lists instead of (void).Bruce Mitchener2015-08-041-6/+6
| | | | | | | | | | | | Summary: This brings the code more in line with the usual LLDB style. NFC. Reviewers: abidh, ki.stfu Subscribers: lldb-commits Differential Revision: http://reviews.llvm.org/D11746 llvm-svn: 243967
* [lldb-mi] Correct file names in first line comments.Bruce Mitchener2015-07-211-1/+1
| | | | llvm-svn: 242781
* Remove typedefs for MIchar, MIschar, MIuchar.Bruce Mitchener2015-07-031-5/+5
| | | | | | | | | | | | | | | | Summary: This is a start on bringing lldb-mi more in line with the typical LLDB coding style. This just removes the usage of the typedefs and doesn't yet clean up any logic or other issues. (This is to keep the review simple.) Reviewers: abidh, ki.stfu, domipheus Subscribers: lldb-commits Differential Revision: http://reviews.llvm.org/D10917 llvm-svn: 241349
* Refactor lldb-mi's promptIlia K2015-05-131-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Summary: This patch fixes/cleans code around of (gdb) prompt: # Add CMICmnStreamStdout::WritePrompt # Clean up CMICmnStreamStdout::TextToStdout (don't lock the m_mutex twice because it will be locked in CMICmnStreamStdout::WritePriv) # Remove unused CMICmnStreamStdin::m_bShowPrompt field # Refactor CMICmnLLDBDebuggerHandleEvents to use CMICmnStreamStdout::WritePrompt instead of TextToStdout("(gdb)") # Refactor CMIDriver to use CMICmnStreamStdout::WritePrompt instead of ``` if (bOk && m_rStdin.GetEnablePrompt()) bOk = m_rStdOut.WriteMIResponse(m_rStdin.GetPrompt()); ``` Test Plan: ./dotest.py -v --executable $BUILDDIR/bin/lldb tools/lldb-mi/ Reviewers: abidh Reviewed By: abidh Subscribers: lldb-commits, abidh Differential Revision: http://reviews.llvm.org/D9737 llvm-svn: 237248
* Improve handling of ctrl-c with MSVC.Hafiz Abid Qadeer2015-04-271-1/+13
| | | | | | | | | Currently hitting Ctrl-C in Windows LLDB-MI just exits MI. But according to test_lldbmi_stopped_when_interrupt() in TestMiSignal.py Ctrl-C should send a SIGINT signal to the inferior. Patch adds this functionality to Windows so SIGINT is sent on Ctrl-C. Patch from EwanCrawford. Reviewed in http://reviews.llvm.org/D9248. llvm-svn: 235887
* Fix infinite loop when ^D was pressed (MI)Ilia K2015-03-261-1/+7
| | | | llvm-svn: 233278
* Remove redundant comments from lldb-mi source files.Hafiz Abid Qadeer2015-03-161-12/+0
| | | | | | | Most of lldb-mi files have comments about environement, copyright etc which were neither needed nor uptodate. This commit removes those comments. llvm-svn: 232396
* Remove redundant code from lldb-mi.Hafiz Abid Qadeer2015-02-241-250/+5
| | | | | | | | | | | | | | | | | | | | | | | | Summary: After recent changes, some code has become redundant. This revision tries to remove the un-used code and tidy up the rest. Following 4 files have been removed. I have updated CMake files and checked that it builds fine on Linux and Windows. Can somebody update the xcode related file accordingly? tools/lldb-mi/MICmnStreamStdinLinux.cpp tools/lldb-mi/MICmnStreamStdinLinux.h tools/lldb-mi/MICmnStreamStdinWindows.cpp tools/lldb-mi/MICmnStreamStdinWindows.h Reviewers: clayborg, ki.stfu Reviewed By: clayborg, ki.stfu Subscribers: lldb-commits Differential Revision: http://reviews.llvm.org/D7834 llvm-svn: 230345
* Reduce number of threads in lldb-mi.Hafiz Abid Qadeer2015-02-201-1/+33
| | | | | | | | | | | | | | | | | | | | | LLDB-mi have 3 threads. 1. Wait for input. 2. Process commands. 3. Process events. This revision merges 1 & 2. Same thread waits on input and then process the command. This way, no synchronization is needed between first and 2nd. Also it is easy to check when to exit. A lot of code will redundant and will be cleaned up gradually. All lldb-mi tests pass with gcc and clang as test compiler. Also did minimal testing on command line and works ok. The "quit" and "-gdb-exit" command close the application without needing any further return. Reviewed in http://reviews.llvm.org/D7746. llvm-svn: 230003
* Fix test_lldbmi_prompt test by enabling ShowPrompt in CMICmnStreamStdin by ↵Ilia K2015-02-131-1/+1
| | | | | | default llvm-svn: 229137
* Replace ioctl with select to reduce processor usage by lldb-mi on OSX.Hafiz Abid Qadeer2014-12-151-0/+12
| | | | | | | | | | | | | | | | This saga started with a hang on OSX. 2 solutions were proposed. 1) 'select' based solution works ok on OSX but slows down test completion time on Linux many times. 2) 'ioctl' base solution also works but it causes heavy processor usage on OSX as reported by Ilia K. But as the original hang did not occur on Linux so this commit re-introduces the 'select' in conditional code so that it only runs for OSX. There is no need for this 'fix' to run on Linux. Initial patch by Ilia K <ki.stfu@gmail.com>. A few changes were made by me. llvm-svn: 224258
* Remove unnecessary changes committed in r223222.Hafiz Abid Qadeer2014-12-031-12/+0
| | | | | | | | | There were 2 different patches in discussion. One using ioctl and other using select. We decided to use the ioctl but committed code also have some changes which were only needed for 'select'. This patch removes them. llvm-svn: 223227
* Fix a hang on OSX while executing -exec-run.Hafiz Abid Qadeer2014-12-031-0/+13
| | | | | | | | | | Now we wait for input to become available before blocking in fgets. More details on problem can be found in http://lists.cs.uiuc.edu/pipermail/lldb-commits/Week-of-Mon-20141201/014290.html Patch from dawn@burble.org. llvm-svn: 223222
* Reformat lldb-mi using clang-format.Zachary Turner2014-11-171-283/+298
| | | | | | Courtesy of dawn@burble.org. llvm-svn: 222150
* Restore some reverted fixes for spelling, newlines and __linux__ macro.Deepak Panickal2014-08-081-5/+5
| | | | llvm-svn: 215234
* Add new MI commands, features and fixes to the lldb-mi driver.Deepak Panickal2014-08-081-5/+5
| | | | | | | | | | - Can now load an executable directly as an argument. - Fixes towards supporting local debugging. - Fixes for stack-list-arguments, data-evaluate-expression, environment-cd, stack-list-locals, interpreter-exec. - Fix breakpoint event handling. - Support dynamic loading of libraries using the search paths provided by Eclipse. llvm-svn: 215223
* Fix a few typos.Bruce Mitchener2014-06-271-5/+5
| | | | llvm-svn: 211851
* Added support for new MI commands and bug fixes. More details in MIReadme.txt.Deepak Panickal2014-06-241-132/+72
| | | | llvm-svn: 211607
* Remove include of obsolete stropts.h headerEd Maste2014-05-261-1/+0
| | | | | | | The header is for POSIX streams functionality, and does not exist on FreeBSD, OS X, or contemporary Linux distributions. llvm-svn: 209632
* Initial commit of LLDB Machine Interface Frontend.Deepak Panickal2014-05-161-0/+482
- Tested with Eclipse, likely to work with other GDB/MI compatible GUIs. - Some but not all MI commands have been implemented. See MIReadme.txt for more info. - Written from scratch, no GPL code, based on LLDB Public API. - Built for Linux, Windows and OSX. Tested on Linux and Windows. - GDB/MI Command Reference, https://sourceware.org/gdb/onlinedocs/gdb/GDB_002fMI.html llvm-svn: 208972
OpenPOWER on IntegriCloud