summaryrefslogtreecommitdiffstats
path: root/lldb/tools/lldb-mi/MICmnLogMediumFile.cpp
Commit message (Collapse)AuthorAgeFilesLines
* [LLDB] Remove lldb-miJonas Devlieghere2019-07-181-385/+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-23/+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
* *** This commit represents a complete reformatting of the LLDB source codeKate Stone2016-09-061-208/+192
| | | | | | | | | | | | | | | | | | | | | | | *** 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
* Typo corrections identified by codespellEd Maste2016-07-191-1/+1
| | | | | | | | | Submitted by giffunip@yahoo.com; I fixed a couple of nearby errors and incorrect changes in the patch. llvm.org/pr27634 llvm-svn: 275983
* [lldb-mi] Fix failure in log file initialization.Bruce Mitchener2015-09-111-0/+1
| | | | | | I broke this in r247388. llvm-svn: 247390
* [lldb-mi] Remove MIUtilSystem*.Bruce Mitchener2015-09-111-24/+7
| | | | | | | | | | | | | | | | | | Summary: This platform-specific code wasn't fully implemented and wasn't actually needed. There was one call for the log file path and that has been addressed. This lets us also remove an error message from MICmnLogMediumFile as it is no longer used. Reviewers: ki.stfu, domipheus, abidh Subscribers: lldb-commits Differential Revision: http://reviews.llvm.org/D12764 llvm-svn: 247388
* [lldb-mi] Use empty arg lists instead of (void).Bruce Mitchener2015-08-041-15/+15
| | | | | | | | | | | | 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] Fix typosBruce Mitchener2015-07-071-2/+2
| | | | llvm-svn: 241585
* [lldb-mi] Use size_t where appropriate.Bruce Mitchener2015-07-031-6/+6
| | | | | | | | | | | | | | | Summary: Many places should have been using size_t rather than MIuint or MIint. This is particularly true for code that uses std::string::find(), std::string::rfind(), std::string::size(), and related methods. Reviewers: abidh, ki.stfu, domipheus Subscribers: lldb-commits Differential Revision: http://reviews.llvm.org/D10931 llvm-svn: 241360
* Remove typedefs for MIchar, MIschar, MIuchar.Bruce Mitchener2015-07-031-3/+3
| | | | | | | | | | | | | | | | 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
* MI fix allowing multiple logging instances of lldb-mi to run simultaneously.Ilia K2015-04-231-15/+28
| | | | | | | | | | | | | | | | | | | | Summary: Currently if two instances of lldb-mi are running with logging enabled using '--log' the log file conflicts. This produces the following error MI: Error: File Handler. Error Permission denied opening 'C:\Users\Ewan\LLVM\build\Debug\bin\lldb-mi-log.txt' Fixed in this patch by renaming lldb-mi-log.txt based on the date, e.g. lldb-mi-log.txt-20150316163631.log, and moving the file into the temp directory by using the --log-dir option. Regrading previous review comments the P_tmpdir macro is defined in Windows but always points to "\", which doesn't help much. Also when using the Windows API for GetTempPath() dynamic memory seems much more messy. Patch from ewan@codeplay.com Reviewers: abidh, EwanCrawford Subscribers: zturner, lldb-commits, deepak2427 Differential Revision: http://reviews.llvm.org/D9054 llvm-svn: 235589
* Changed a function to static.Hafiz Abid Qadeer2015-03-311-1/+1
| | | | | | | | A temp object was being created to call StripOffFileName. This function is not dependent on class members so I am making it static. No regression on testsuite on Linux. llvm-svn: 233703
* 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
* Fix the LLDB build under Debian KfreebsdSylvestre Ledru2015-02-101-1/+1
| | | | | | | | | | | | | | Summary: I don't know if there is a better way for the change in source/Host/freebsd/ThisThread.cpp Reviewers: emaste Subscribers: hansw, emaste, lldb-commits Differential Revision: http://reviews.llvm.org/D7441 llvm-svn: 228710
* Reformat lldb-mi using clang-format.Zachary Turner2014-11-171-285/+305
| | | | | | Courtesy of dawn@burble.org. llvm-svn: 222150
* Restore some reverted fixes for spelling, newlines and __linux__ macro.Deepak Panickal2014-08-081-1/+1
| | | | llvm-svn: 215234
* Add new MI commands, features and fixes to the lldb-mi driver.Deepak Panickal2014-08-081-8/+2
| | | | | | | | | | - 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
* Use __linux__ macro throughout, instead of ocasional __linux.Sylvestre Ledru2014-07-301-1/+1
| | | | | | | | | | | | __linux is not universally defined across all standards versions, compilers and architectures. Specifically at C99 and up, it's not defined in GCC on powerpc platform. See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=28314 Bugzilla: http://llvm.org/bugs/show_bug.cgi?id=20380 Patch by Dimitri John Ledkov <dimitri.ledkov@canonical.com> llvm-svn: 214291
* Restore fixes reverted by r211607:Ed Maste2014-06-241-1/+1
| | | | | | | | | | | | | | | | | | | r209631: Use MIUtilSystemLinux on FreeBSD as well We should later rename this file (probably MIUtilSystemPOSIX), but more clean-up is still needed here, and we can wait until we better understand how this code may be shared between FreeBSD, Linux, and OS X. r209632: Add stdlib.h for malloc and friends r209633: Remove include of obsolete stropts.h header The header is for POSIX streams functionality, and does not exist on FreeBSD, OS X, or contemporary Linux distributions. Issue reported by John Wolfe. llvm-svn: 211620
* Added support for new MI commands and bug fixes. More details in MIReadme.txt.Deepak Panickal2014-06-241-1/+1
| | | | llvm-svn: 211607
* Use MIUtilSystemLinux on FreeBSD as wellEd Maste2014-05-261-1/+1
| | | | | | | | We should later rename this file (probably MIUtilSystemPOSIX), but more clean-up is still needed here, and we can wait until we better understand how this code may be shared between FreeBSD, Linux, and OS X. llvm-svn: 209631
* Initial commit of LLDB Machine Interface Frontend.Deepak Panickal2014-05-161-0/+426
- 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