Merge pull request #13 from a-okoshkin/refactoring

Minor refactoring

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

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.

Added `pragma once` to cputil.h

Redesign of `to_utf8()`

* String length is not passed any more due to it's buggy implementation.
* New overloaded version for a single wide-char is added.
* `WCHAR` is not necessary to use in this header as `char16_t` has been already
passed to the `std::codecvt_utf8_utf16` template.

Use `dotnet` CLI to launch application

Dynamically load PAL symbols from libcoreclr instance

Now the debugger depends only on libdbgshim.

Add DLOpen and DLSym wrappers

Add `USleep` function (based in `usleep`) instead of `Sleep` from PAL

Use `std::wstring_convert()` instead of PAL-based conversion

Merge pull request #7 from a-okoshkin/refactoring

Minor code refactoring

Merge pull request #6 from d-kovalenko/upd_patches

Update patches for coreclr

`m_clrPath` is initialized in ManagedDebugger::AttachToProcess()

Previously `m_clrPath` was a local variable.

`GetCLRPath()` and `InternalEnumerateCLRs()` accept `pid` as `DWORD`

because underlying dbgshim API handles PID as `DWORD`.

`pid` is `DWORD` in Startup() and AttachToProcess() from ManagedDebugger

dbgshim API handles PID as `DWORD`.

Update patches for coreclr

patches for review.tizen.org:29418/platform/upstream/coreclr:58c98ea

Signed-off-by: Dmitry Kovalenko <d.kovalenko@samsung.com>

Debugger::IsJustMyCode() is const now

Make EmitInitializedEvent() and EmitTerminatedEvent() one-line in VSCodeProtocol

Add missing initialisation of ManagedDebugger fields

* `m_lastStoppedThreadId` is set to 1
* `m_protocol` is set to nullptr

MIProtocol and VSCodeProtocol default constructors initialise m_debugger

MIProtocol::m_debugger and VSCodeProtocol::m_debugger are initialised as nullptr.

Drop extra ';' in MIProtocol header

EmitInitializedEvent() and EmitTerminatedEvent() have empty implementation.

Merge pull request #5 from a-okoshkin/improve_cmake

Improve CMake scripts

Merge pull request #4 from i-kulaychuk/enable-stopatentry

Add support for stopAtEntry option

Merge pull request #3 from i-kulaychuk/enable-logging

Add support for --engineLogging command line option

Always enable stopAtEntry when launching through MI protocol

Remove unused parameter

Add entry event in MI protocol

Add support for --engineLogging command line option

Defines related to the build target are properly set now

Target-related defines are set based on `CLR_CMAKE_TARGET_ARCH_*` variables
but not `CLR_CMAKE_PLATFORM_ARCH_*`.
Platform-related defines are set based on now `CLR_CMAKE_PLATFORM_ARCH_*`
but not `CLR_CMAKE_TARGET_ARCH_*`.
Now it's not necessary to explicitly set `CLR_CMAKE_TARGET_ARCH_*=1` for Linux.
Copied from CoreCLR CMake scripts.

CoreCLR libraries are imported first in `netcoredbg/CMakeLists.txt`

Beautification: the script is logically divided into parts now.

Add missing `clr_unknown_arch()` to clrdefinitions.cmake

Add default value for `CLR_BIN_DIR` cmake variable

By default it's equal to `CLR_DIR`.

Add support for stopAtEntry option

Compile only specified files when building SymbolReader

Merge pull request #2 from a-okoshkin/fix_cmake_managed_build

Add support of out-of-source SymbolReader build

Add support of out-of-source SymbolReader build

Custom CMake `dotnet build` command respects `CMAKE_CURRENT_BINARY_DIR`.

Fix OutputEvent body

Initial support for VSCode debugger protocol

Report number of available frames and use levels instead of high frame index

Move ManagedDebugger class declaration into a separate header

Add Continued event and clear variable references when this event happens

Refactor Debugger - make initialization more verbose

Fix frame id

Move JMC related methods into Modules class

Refactor printing loaded libraries (modules)

Rename commands.cpp to miprotocol.cpp

Move all MIProtocol methods implementations into commands.cpp

Refactor ManagedDebugger implementation into separate file

Refactor Debugger into the ManagedDebugger class which implements Debugger interface

Refactor variables handler into Variables class

Refactor breakpoints handler into the Breakpoints class

Do not allow access to private fields of ManagedCallback from Debugger

Make GetThreadsState function non-static

Refactor Modules namespace into Modules class

Make PrintFrameLocation a private static method of the MIProtocol class

Make DisableAllSteppers a private static method of the Debugger class

Refactor last stopped thread id global variable

Refactor global process variable

Move all Evaluator methods implementations into expr.cpp and valuewalk.cpp

Refactor all evaluation related functions into separate Evaluator class

Match only full paths when setting a breakpoint

Refactor internal structure for handling breakpoints

Make methods SetJustMyCode/IsJustMyCode non-static

Add virtual destructor to Protocol interface

Construct ManagedCallback class instance inside Debugger constructor

Rename varobj.cpp to variables.cpp

Move class constructor related global variables into Debugger class