Remove useless code

Do no call virtual functions from destructor

Merge pull request #30 from dotnet/redirect-output

Redirect output

Refactor IORedirectServer

Enable capturing stdout/stderr of launched application

Fix OutputEvent category

Merge pull request #29 from dotnet/fix-crossplatform-build

Fix crossplatform build issues

Fix lambda capture warnings

Reorder includes to avoid redifinition warnings in PAL

PAL headers should come before any standard C++ headers.

Remove UNICODE redifinition

Do not use PAL_CPP_TRY/PAL_CPP_CATCH_ALL

Enable better search for CoreCLR bin directory

Automatically find dotnet utility

Update cmake files from CoreCLR

Fix uninitialized frame address

Merge pull request #28 from dotnet/cleanup-readme

Cleanup README

Automaticaly find CoreCLR bin directory if it is not specified

Simplify build parameters and update README

Remove patches directory

Merge pull request #25 from dotnet/fix-break-all-in-vs

Fix Break All behavior in Visual Studio and Visual Studio Code

Do not report invalid Source in JSON

Fix uninitialized Frame fields

Fix Break All behavior in Visual Studio

Merge pull request #26 from dotnet/unset-core-profiler-vars

Unset profiler env var for debugger own libcoreclr

Unset profiler env var for debugger own libcoreclr

Merge pull request #23 from i-kulaychuk/vscode-server

Improvements for Tizen and VS plugin integration

Make arguments after '--' set (MI) or override (VSCode) launch command

Close stdin, stdout and stderr when using --server option

Return E_NOTIMPL from all unimplemented ManagedCallback methods

Add ability to launch application from command line arguments (after --)

Print MI output through std::cout instead of stdout

Add option for listening requests on the TCP/IP port instead of stdin/out

Add file name without path to MI breakpoint output

Use dynamic input buffer for MI commands

Merge pull request #22 from a-aksenov/refactor-protocol__27_04_18

Refactor Protocol

Refactor MIProtocol

Make non-interface methods private

Signed-off-by: Alexander Aksenov <a.aksenov@samsung.com>

Move Protocol's derived classes commons in Protocol

Signed-off-by: Alexander Aksenov <a.aksenov@samsung.com>

Merge pull request #21 from i-kulaychuk/fix-breakpoints

Fix breakpoints

Fix breakpoint removal

Add test for adding/removing breakpoints

Fix ids of fake exception breakpoints

Merge pull request #17 from i-kulaychuk/add-exception-test

Add ExceptionTest

Increase default test Expect timeout to 20 seconds

Merge pull request #20 from a-aksenov/fix_for_arch

Add missing header for modules.h

Add missing header for modules.h

Signed-off-by: Alexander Aksenov <a.aksenov@samsung.com>

Merge pull request #18 from i-kulaychuk/tizen-fixes

Tizen+MIEngine fixes

Handle breakpoints without full file path

Fix MI message output

Avoid crashing when funceval does not work

Wait for coreclr to load (3s timeout) when attaching to process

Fix handling module paths with /proc/self prefix on Tizen

Fix output of MI `-thread-info` command

Always report nearest sequence point to fix ExceptionTest

Add ExceptionTest

Merge pull request #16 from i-kulaychuk/debugger-tests

Debugger tests

Always close test process after performing a test

Fix misspellings and assembly references

Merge pull request #13 from a-okoshkin/refactoring

Minor refactoring

Update tests README

Update tests

Renamed tests.
Now it is required to specify test names in runner project.
Tests solution moved to tests directory.
Add ability to run debugger through custom pipe script.
Add script for running tests through sdb.

Add option for disabling libdbgshim installation

Use only output string stream instead of the input-output one

Enable handling  MI status

Add test solution

Copy libdbgshim.so to install directory

Initial files for xunit-based tests

Refactoring of ManagedDebugger::RunProcess()

Move static `SymbolReaderDllName` and `SymbolReaderClassName` out of the header

Fixe size type cast (sized-unsized) in EscapeShellArg()

Drop unused variable in DisableAllBreakpointsAndSteppersInAppDomain()

Merge pull request #14 from a-okoshkin/change_cordebug_version

Use `CorDebugVersion_4_0` in `CreateDebuggingInterfaceFromVersionEx()`

Use `CorDebugVersion_4_0` in `CreateDebuggingInterfaceFromVersionEx()`

It was `CorDebugLatestVersion` previously.

Fix gbs build

Merge pull request #10 from a-okoshkin/fix_header_inclusion

Fix inclusion of some headers

Drop `common.h` as unused

Remove `common.h` header inclusion from `main.cpp`

Remove unnecessary header inclusions

Fixed header inclusions in `miprotocol.h`

Fixed header inclusions in `vscodeprotocol.h`

Fixed header inclusions in `valueprint.h`

Add missing header inclusions in `symbolreader.h`
Note that currently `TADDR` is not defined in `symbolreader.h` and
variables of this type are not used.

Merge pull request #11 from a-okoshkin/refactoring

Minor refactoring

Fixed header inclusions in `frames.h`

Fixed header inclusions in `typeprinter.h`

Fixed header inclusions in `manageddebugger.h`

Add missing inclusion of headers to modules.h

`pragma once` is added to `modules.h`.

Add missing inclusion of headers to debugger.h

Add missing `pragma once` to `torelease.h`

Make ParseGenericParams() and FindTypeInModule() static in expr.cpp

These functions are for local use only.

Fixed header inclusion for `platform.h`

Get rid of `cmdString` temporary variable in ManagedDebugger::RunProcess()

Use `nullptr` where it's possible

DereferenceAndUnboxValue(), PrintEnumValue(), PrintValue() and
Evaluator::WalkMembers() use `nullptr` instead of `NULL`.

Drop `-DCLR_CMAKE_TARGET_ARCH_AMD64=1` cmake option from README

Now it's not necessary to specify it for Ubuntu build.

Update `.gitignore`

* Add Qt Creator files
* Add build artifacts

Use `nullptr` everywhere in `class ToRelease`

Do not mix usage of `NULL` and `nullptr`.

Use `enum CorDebugInterfaceVersion` in CreateDebuggingInterfaceFromVersionEx()

Do not use magic numbers.
`CorDebugLatestVersion` (=`CorDebugVersion_4_5` is 5 for now) can be used
instead of current equivalent (4) of `CorDebugVersion_4_0`.

Merge pull request #12 from i-kulaychuk/wip-no-pal

Remove dependency on libmscordaccore, use PAL from libcoreclr instance

Merge pull request #9 from a-okoshkin/fix_wchar_convert

Minor improvement of wchar convertion

Add `to_utf16()` instead of `convert.from_bytes()`

Evaluator::GetFieldOrPropertyWithName(), ManagedDebugger::RunProcess() and
ManagedDebugger::AttachToProcess() use `to_utf16()` now.

Add missing usage of `to_utf8()` instead of `convert.to_bytes()`

Evaluator::WalkStackVars() and SymbolReader::LoadSymbolsForPortablePDB() use
`to_utf8()` now.