use wx for clipboard handling; slight refactor

.github/workflows/sonarcloud.yaml

@@ -3,6 +3,7 @@ on:
   push:
     branches:
       - "master"
+      - "2.0"
 
 jobs:
   "sonarcloud":

src/Cpu6502.cpp

@@ -1,7 +1,7 @@
-/* 
+/*
  * File:   Cpu6502.cpp
  * Author: Christopher
- * 
+ *
  * Created on December 12, 2013, 10:14 PM
  */
 
@@ -33,6 +33,7 @@ void Cpu6502::clock(bool phase) {
     setPins(PinSettings{std::make_pair(common.CLK0,phase)});
     rw();
 
+    // TODO turn these into trace log settings
 #ifdef TRACEREG
     this->trace.dumpRegisters();
 #endif

src/E2wxApp.cpp

@@ -149,11 +149,12 @@ bool E2wxApp::OnInit() {
 //    wxLogWarning("%s", "a warning has occurred");
 //    wxLogInfo("%s", "informational");
 //    wxLogVerbose("%s", "verbose");
-//    wxFile().Open("foobar.txt");
+//    wxFile().Open("foobar.txt"); // <-- example of generating wx log message
 
 //    TODO wx log file window?
 //    TODO remove all logging from stdout/err to log file
 //    TODO define components to turn on/off for trace level logging (disk, cassette, woz, keyboard, ram, lss, audio, etc.)
+//    TODO why are log messages getting buffered? (It ruins the timestamp.)
 
     BOOST_LOG_TRIVIAL(info) << "Application ID: " << this->GetID();
     BOOST_LOG_TRIVIAL(info) << "Application version: " << this->GetVersion();

src/PreferencesDialog.cpp

@@ -33,7 +33,7 @@ wxBEGIN_EVENT_TABLE(PreferencesDialog, wxDialog)
     EVT_BUTTON(XRCID("btnDelete"), PreferencesDialog::OnDelete)
     EVT_BUTTON(XRCID("btnRename"), PreferencesDialog::OnRename)
 wxEND_EVENT_TABLE()
-
+// TODO import external config file
 
 
 

src/clipboardhandler.cpp

@@ -17,22 +17,25 @@
 */
 #include "clipboardhandler.h"
 
-#include <SDL.h>
+#include <wx/clipbrd.h>
-#include <limits.h>
 
-ClipboardHandler::ClipboardHandler()
+
-{
+ClipboardHandler::ClipboardHandler() {
 }
 
-ClipboardHandler::~ClipboardHandler()
+ClipboardHandler::~ClipboardHandler() {
-{
 }
 
-std::string ClipboardHandler::getText()
+wxString ClipboardHandler::getText() {
-{
+    wxString ret;
-    std::string ret;
+    if (wxTheClipboard->Open())
-        char* sdlAllocatedText = SDL_GetClipboardText();
+    {
-        ret.assign(sdlAllocatedText);
+        if (wxTheClipboard->IsSupported(wxDF_TEXT)) {
-        SDL_free(sdlAllocatedText);
+            wxTextDataObject data;
+            wxTheClipboard->GetData(data);
+            ret = data.GetText();
+        }
+        wxTheClipboard->Close();
+    }
     return ret;
 }

src/clipboardhandler.h

@@ -18,16 +18,14 @@
 #ifndef CLIPBOARDHANDLER_H
 #define CLIPBOARDHANDLER_H
 
-#include <string>
+#include <wx/string.h>
 
-
+class ClipboardHandler {
-class ClipboardHandler
-{
 public:
     ClipboardHandler();
     ~ClipboardHandler();
 
-    std::string getText();
+    wxString getText();
 };
 
 #endif

src/emulator.cpp

@@ -22,6 +22,7 @@
 #include "e2const.h"
 
 #include <wx/msgdlg.h>
+#include <wx/string.h>
 
 #include <SDL.h>
 
@@ -363,16 +364,13 @@ void Emulator::powerOffComputer() {
 
 void Emulator::paste() {
     // Feed input from the clipboard to the Apple keyboard
-    std::string s = this->clip.getText();
+    wxString s = this->clip.getText();
-    for (unsigned int i = 0; i < s.length(); ++i) {
+    s.Replace("\n\r", "\r\n");
-        unsigned char key = s[i];
+    s.Replace("\r\n", "\r");
-        // TODO fix pasting line-endings
+    s.Replace("\n", "\r");
-        if (key == '\n')
+    s.MakeUpper();
-            key = '\r';
+    for (auto c : s) {
-        if ('a' <= key && key <= 'z') {
+        this->keypresses.push(c);
-            key -= 32;
-        }
-        this->keypresses.push(key);
     }
 }
 

src/keyboard.h

@@ -18,8 +18,8 @@
 #ifndef KEYBOARD_H
 #define KEYBOARD_H
 
-#include <SDL.h>
 #include <queue>
+#include <cstdint>
 
 typedef std::queue<unsigned char> KeypressQueue;
 
@@ -31,7 +31,7 @@ class Keyboard {
 
     unsigned char latch;
     unsigned char cGet;
-    Uint32 lastGet;
+    uint32_t lastGet;
 
 public:
     Keyboard(KeypressQueue& q, KeyboardBufferMode& buffered);