Skip to content

Zcooger patch for ctrl+c viewport#202

Open
Zcooger wants to merge 7 commits intooyvindln:vhs_decodefrom
Zcooger:Zcooger-patch-ctrl-c-viewport
Open

Zcooger patch for ctrl+c viewport#202
Zcooger wants to merge 7 commits intooyvindln:vhs_decodefrom
Zcooger:Zcooger-patch-ctrl-c-viewport

Conversation

@Zcooger
Copy link
Collaborator

@Zcooger Zcooger commented Mar 20, 2025

Easy sharing feature, please test as I'm not sure it will work.

@Zcooger Zcooger self-assigned this Mar 20, 2025
@harrypm
Copy link
Collaborator

harrypm commented Mar 23, 2025

This should be upstream for ld-analyse?

@Zcooger
Copy link
Collaborator Author

Zcooger commented Mar 23, 2025

It only needs merging with vectorscope commit happycube@bb3b715

@harrypm
Copy link
Collaborator

harrypm commented Jun 18, 2025

Was this build tested on your end @Zcooger ?

All I am getting is:

[ 13%] Building CXX object tools/ld-analyse/CMakeFiles/ld-analyse.dir/mainwindow.cpp.o
/home/harry/vhs-decode/tools/ld-analyse/mainwindow.cpp: In member function ‘void MainWindow::setGuiEnabled(bool)’:
/home/harry/vhs-decode/tools/ld-analyse/mainwindow.cpp:168:9: error: ‘class Ui::MainWindow’ has no member named ‘actionCopy_frame_to_clipboard’
  168 |     ui->actionCopy_frame_to_clipboard->setEnabled(enabled);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/harry/vhs-decode/tools/ld-analyse/mainwindow.cpp: In member function ‘void MainWindow::on_actionCopy_frame_to_clipboard_triggered()’:
/home/harry/vhs-decode/tools/ld-analyse/mainwindow.cpp:747:30: error: qualified-id in declaration before ‘(’ token
  747 |     void QClipboard::setImage(const QImage &imageToCopy, QClipboard::Mode mode = Clipboard)
      |                              ^
make[2]: *** [tools/ld-analyse/CMakeFiles/ld-analyse.dir/build.make:164: tools/ld-analyse/CMakeFiles/ld-analyse.dir/mainwindow.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:1420: tools/ld-analyse/CMakeFiles/ld-analyse.dir/all] Error 2
make: *** [Makefile:146: all] Error 2

@Zcooger
Copy link
Collaborator Author

Zcooger commented Jun 18, 2025

I'll check that. I had no build env setup yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants