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.

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