Reworked all of PADRE to use Tokio 1.0 with superior architecture.

Added LLDB as a full debugger.

Full reworking, too much detail to go into here.

Fixed all tests apart from python/node based stuff which will come in a
later commit.

Added Python debugger

Author: Steven Trotter

Makefile

@@ -1,2 +1,5 @@
 all:
 	cd padre && cargo build
+
+clean:
+	cd padre && cargo clean

autoload/padre/debugger.vim

@@ -135,7 +135,7 @@ function! padre#debugger#Debug(...)
 endfunction
 
 function! padre#debugger#Run()
-  call padre#socket#Send({"cmd": "run"}, function('padre#debugger#RunCallback'))
+  call padre#socket#Send({"cmd": "run"}, function('padre#debugger#GenericCallback'))
 endfunction
 
 function! padre#debugger#Stop()
@@ -161,7 +161,7 @@ function! padre#debugger#Stop()
 endfunction
 
 function! s:SetBreakpointInDebugger(line, file)
-  call padre#socket#Send({"cmd": "breakpoint", "file": a:file, "line": str2nr(a:line)}, function('padre#debugger#BreakpointCallback'))
+  call padre#socket#Send({"cmd": "breakpoint", "file": a:file, "line": str2nr(a:line)}, function('padre#debugger#GenericCallback'))
 endfunction
 
 function! padre#debugger#Breakpoint()
@@ -173,19 +173,19 @@ function! padre#debugger#Breakpoint()
 endfunction
 
 function! padre#debugger#StepIn()
-  call padre#socket#Send({"cmd": "stepIn"}, function('padre#debugger#StepInCallback'))
+  call padre#socket#Send({"cmd": "stepIn"}, function('padre#debugger#GenericCallback'))
 endfunction
 
 function! padre#debugger#StepOver()
-  call padre#socket#Send({"cmd": "stepOver"}, function('padre#debugger#StepOverCallback'))
+  call padre#socket#Send({"cmd": "stepOver"}, function('padre#debugger#GenericCallback'))
 endfunction
 
 function! padre#debugger#PrintVariable(variable)
   call padre#socket#Send({"cmd": "print", "variable": a:variable}, function('padre#debugger#PrintVariableCallback'))
 endfunction
 
 function! padre#debugger#Continue()
-  call padre#socket#Send({"cmd": "continue"}, function('padre#debugger#ContinueCallback'))
+  call padre#socket#Send({"cmd": "continue"}, function('padre#debugger#GenericCallback'))
 endfunction
 
 """"""""""""""""
@@ -198,63 +198,12 @@ function! padre#debugger#SignalPADREStarted()
   endfor
 endfunction
 
-function! padre#debugger#RunCallback(channel_id, data)
-  if a:data['status'] != 'OK'
-    call padre#debugger#Log(2, 'Error: ' . string(a:data))
-    return
-  endif
-
-  if has_key(a:data, 'pid')
-    call padre#debugger#Log(4, 'Process ' . a:data['pid'] . ' Running')
-  endif
-endfunction
-
-function! padre#debugger#BreakpointCallback(channel_id, data)
-  if a:data['status'] == 'OK'
-  elseif a:data['status'] == 'PENDING'
-    call padre#debugger#Log(4, 'Breakpoint pending')
-  else
-    call padre#debugger#Log(2, 'Error: ' . string(a:data))
-  endif
-endfunction
-
-function! padre#debugger#BreakpointSet(fileName, lineNum)
-  let l:msg = 'Breakpoint set file=' . a:fileName . ', line=' . a:lineNum
-  call padre#debugger#Log(4, l:msg)
-endfunction
-
-function! padre#debugger#StepInCallback(channel_id, data)
+function! padre#debugger#GenericCallback(channel_id, data)
   if a:data['status'] != 'OK'
     call padre#debugger#Log(2, 'Error: ' . string(a:data))
   endif
 endfunction
 
-function! padre#debugger#StepOverCallback(channel_id, data)
-  if a:data['status'] != 'OK'
-    call padre#debugger#Log(2, 'Error: ' . string(a:data))
-  endif
-endfunction
-
-function! padre#debugger#ContinueCallback(channel_id, data)
-  if a:data['status'] != 'OK'
-    call padre#debugger#Log(2, 'Error: ' . string(a:data))
-  endif
-endfunction
-
-function! padre#debugger#PrintVariableCallback(channel_id, data)
-  let l:status = remove(a:data, 'status')
-  if l:status != 'OK'
-    call padre#debugger#Log(2, 'Error printing variable: ' . string(a:data))
-    return
-  endif
-
-  let l:variable_name = remove(a:data, 'variable')
-
-  execute "let l:json = system('python -m json.tool', '" . substitute(json_encode(a:data), "'", "''", "g") . "')"
-  let l:msg = 'Variable ' . l:variable_name . "=\n" . l:json
-  call padre#debugger#Log(4, l:msg)
-endfunction
-
 function! padre#debugger#JumpToPosition(file, line)
   let l:msg = 'Stopped file=' . a:file . ' line=' . a:line
   call padre#debugger#Log(4, l:msg)
@@ -304,8 +253,35 @@ function! padre#debugger#JumpToPosition(file, line)
   redraw
 endfunction
 
-function! padre#debugger#ProcessExited(exit_code, pid)
-  call padre#debugger#Log(4, 'Process ' . a:pid . ' finished with exit code=' . a:exit_code)
+function! padre#debugger#PrintVariableCallback(channel_id, data)
+  if a:data['status'] != 'OK'
+    call padre#debugger#Log(2, 'Error: ' . string(a:data))
+  endif
+
+  let s:text = 'Variable (' . a:data['type'] . ') ' . a:data['variable'] . '=' . a:data['value']
+
+  let l:current_tabpagenr = tabpagenr()
+
+  call padre#layout#OpenTabWithBuffer('PADRE_Logs_' . s:PadreNumber)
+
+  let l:current_window = winnr()
+  let l:logs_window = padre#layout#GetLogsWindow()
+
+  if l:current_window != l:logs_window
+    execute l:logs_window . ' wincmd w'
+  endif
+
+  call padre#buffer#AppendBuffer(strftime('%y/%m/%d %H:%M:%S ') . s:text, 0)
+
+  if l:current_window != l:logs_window
+    execute l:current_window . ' wincmd w'
+  endif
+
+  if l:current_tabpagenr != tabpagenr()
+    execute l:current_tabpagenr . 'tabnext'
+  endif
+
+  redraw
 endfunction
 
 function! padre#debugger#Log(level, text)

azure-pipelines.yml

@@ -3,13 +3,13 @@ trigger:
 - feature/*
 
 jobs:
-- job: rustfmt
+- job: rust_static
   pool:
     vmImage: 'ubuntu-latest'
   timeoutInMinutes: 5
 
   steps:
-  - template: ci/rustfmt.yml
+  - template: ci/rust-static.yml
 
 - job: padre_tests
   pool:

ci/install-rust.yml

@@ -2,7 +2,8 @@ steps:
   - script: |
       set -e
       curl https://sh.rustup.rs -sSf | sh -s -- -y --default-toolchain stable
-      echo "##vso[task.setvariable variable=PATH;]$PATH:$HOME/.cargo/bin"
+      echo "##vso[task.setvariable variable=PATH;]$HOME/.cargo/bin:$PATH"
+      rustup update
     displayName: Install rust
 
   - script: |

ci/padre-tests.yml

@@ -2,37 +2,16 @@ steps:
   - template: install-rust.yml
 
   - script: |
-      cargo install cargo-junit
-    displayName: Install test reporters
-
-  - script: |
-      sudo apt-get install software-properties-common python3 python3-pip python3-setuptools
-      pip3 install wheel
-      pip3 install behave pyhamcrest psutil
-    displayName: Install python dependencies
-
-  - script: |
-      cd padre
-      cargo test -- --nocapture
-    env:
-      RUST_BACKTRACE: 1
-      RUSTFLAGS: '-D warnings'
-    displayName: Run Padre CLI unit testing
-
-  - script: |
-      cd padre
-      cargo junit --name TESTS-cargo.xml
-    displayName: Run Padre CLI unit testing reporting
-
-  - script: |
-      sudo apt-get install -y lldb-5.0 gcc
+      sudo apt-get install -y software-properties-common python3 python3-pip python3-setuptools lldb-5.0 gcc
       sudo ln -s /usr/bin/lldb-5.0 /usr/bin/lldb
       sudo ln -s /usr/bin/lldb-server-5.0 /usr/bin/lldb-server
       sudo ln -s /usr/bin/lldb-server-5.0 /usr/lib/llvm-5.0/bin/lldb-server-5.0.0
       sudo rm /usr/local/bin/node
       curl -sL https://deb.nodesource.com/setup_12.x | sudo -E bash -
       sudo apt-get install -y nodejs
-    displayName: Install lldb-5, node-12 and build essentials
+      pip3 install wheel
+      pip3 install behave pyhamcrest psutil
+    displayName: Install lldb-5, node-12 and build and test essentials
 
   - script: |
       cd padre

ci/rust-static.yml

@@ -0,0 +1,27 @@
+steps:
+  - template: install-rust.yml
+
+  - script: |
+      cd padre
+      cargo fmt --all -- --check
+    displayName: Check formatting
+
+  - script: |
+      rustup component add rustfmt
+    displayName: Install rustfmt
+
+  - script: |
+      cargo install cargo-junit
+    displayName: Install test reporters
+
+  - script: |
+      cd padre
+      cargo test -- --nocapture
+    env:
+      RUST_BACKTRACE: 1
+    displayName: Run Padre CLI unit testing
+
+  - script: |
+      cd padre
+      cargo junit --name TESTS-cargo.xml
+    displayName: Run Padre CLI unit testing reporting