Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SaveAs hangs with empty window rotating blue... #1191

Closed
3 tasks done
KurtE opened this issue Jul 14, 2022 · 4 comments · Fixed by #1255
Closed
3 tasks done

SaveAs hangs with empty window rotating blue... #1191

KurtE opened this issue Jul 14, 2022 · 4 comments · Fixed by #1255
Assignees
Labels
topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project

Comments

@KurtE
Copy link

KurtE commented Jul 14, 2022

Describe the problem

I was editing one of my throw away sketch (xxx, yyy, zzz) in this case abc.
I decided I wanted to save this one:

So I went to File->SaveAs
It brought up the save as dialog. Although I thought sort of a random starting directory...

I browsed to where I wanted it: /Teensy Tests
And then entered new name: t4_eeprom_test_stress.ino
The dialog went away, it looked like it closed the: abc sketch window
And opened up a new window:

Which has been hung in the state:
image

Note: I did/do have another sketch open in another window:
image

In this case it does look like it did do the save:
image

To reproduce

Steps mentioned above

Expected behavior

I expected it to simply change the title of the open window to the new name.

Alternatively: at least finish opening the window

Arduino IDE version

Nightly 0713

Operating system

Windows

Operating system version

Windows 10

Additional context

I am not sure this might be a duplicate or related to a few others.

Like #1111
Or #1119

But just in case

Issue checklist

  • I searched for previous reports in the issue tracker
  • I verified the problem still occurs when using the latest nightly build
  • My report contains all necessary details
@KurtE KurtE added the type: imperfection Perceived defect in any part of project label Jul 14, 2022
@kittaakos kittaakos added the topic: code Related to content of the project itself label Jul 14, 2022
@kittaakos kittaakos self-assigned this Jul 14, 2022
@kittaakos
Copy link
Contributor

Like #1111
Or #1119

#922?

@KurtE
Copy link
Author

KurtE commented Jul 14, 2022

Yes - somehow my search missed that #922

Feel free to close, unless this scenario gives you additional hints

@kittaakos
Copy link
Contributor

kittaakos commented Jul 15, 2022

For the sake of transparency, I am going to log suspicious issues here and update my comment. We can decide later if this issue is the same as #922.

  • Create a new sketch, trigger save as with Ctrl+Shift+S. Nothing happens.
root WARN Warning! The request sketch path was different than the response sketch path from the CLI. This could be a potential bug. Request: <c:\Users\KITTAA~1\AppData\Local\Temp\.arduinoIDE-unsaved2022615-24388-1lcn4ep.kouh\sketch_jul15a>, response: <c:\Users\kittaakos\AppData\Local\Temp\.arduinoIDE-unsaved2022615-24388-1lcn4ep.kouh\sketch_jul15a>.
  • Create a new sketch, save it, and use spaces in the sketch name. It saves, but I see this in the console:
root ERROR (node:28972) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 destroyed listeners added to [EventEmitter]. Use emitter.setMaxListeners() to increase limit
    at _addListener (node:events:469:17)
    at Object.addListener (node:events:491:10)
    at Object.once (node:events:535:8)
    at ElectronMessagingContribution.handleIpcMessage (C:\Users\kittaakos\dev\arduino-ide\node_modules\@theia\core\lib\electron-main\messaging\electron-messaging-contribution.js:116:20)
    at IpcMainImpl.<anonymous> (C:\Users\kittaakos\dev\arduino-ide\node_modules\@theia\core\lib\electron-main\messaging\electron-messaging-contribution.js:52:18)
    at IpcMainImpl.emit (node:events:394:28)
    at Object.<anonymous> (node:electron/js2c/browser_init:165:9959)
    at Object.emit (node:events:394:28)
    at Object.callbackTrampoline (node:internal/async_hooks:130:17)

  • Default save as path calculation is broken here. Steps: open an existing sketch (directories#user/alma), trigger save as, the default location must be directories#user and name new proposed name must be alma_copy_$date. But it's file:///c%3A/Users/kittaakos/Documents/Arduino/file%3A///c%253A/Users/kittaakos/Documents/Arduino/alma_copy_20220715154718. It's expected to be file:///c%3A/Users/kittaakos/Documents/Arduino/alma_copy_20220715154718

kittaakos pushed a commit to kittaakos/arduino-ide that referenced this issue Jul 15, 2022
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
kittaakos pushed a commit to kittaakos/arduino-ide that referenced this issue Jul 18, 2022
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
kittaakos pushed a commit to kittaakos/arduino-ide that referenced this issue Jul 18, 2022
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
kittaakos pushed a commit that referenced this issue Jul 26, 2022
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
kittaakos pushed a commit that referenced this issue Jul 26, 2022
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
kittaakos pushed a commit that referenced this issue Jul 27, 2022
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
kittaakos pushed a commit that referenced this issue Jul 27, 2022
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
kittaakos pushed a commit that referenced this issue Jul 27, 2022
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
kittaakos pushed a commit that referenced this issue Jul 27, 2022
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
kittaakos pushed a commit that referenced this issue Jul 27, 2022
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
kittaakos pushed a commit that referenced this issue Jul 27, 2022
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
kittaakos pushed a commit that referenced this issue Jul 27, 2022
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
kittaakos pushed a commit that referenced this issue Jul 27, 2022
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
kittaakos pushed a commit that referenced this issue Jul 28, 2022
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
kittaakos pushed a commit that referenced this issue Jul 28, 2022
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
@kittaakos
Copy link
Contributor

  • Create a new sketch, trigger save as with Ctrl+Shift+S. Nothing happens.
root WARN Warning! The request sketch path was different than the response sketch path from the CLI. This could be a potential bug. Request: <c:\Users\KITTAA~1\AppData\Local\Temp\.arduinoIDE-unsaved2022615-24388-1lcn4ep.kouh\sketch_jul15a>, response: <c:\Users\kittaakos\AppData\Local\Temp\.arduinoIDE-unsaved2022615-24388-1lcn4ep.kouh\sketch_jul15a>.

7ff8a2c

  • Create a new sketch, save it, and use spaces in the sketch name. It saves, but I see this in the console:
root ERROR (node:28972) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 destroyed listeners added to [EventEmitter]. Use emitter.setMaxListeners() to increase limit
    at _addListener (node:events:469:17)
    at Object.addListener (node:events:491:10)
    at Object.once (node:events:535:8)
    at ElectronMessagingContribution.handleIpcMessage (C:\Users\kittaakos\dev\arduino-ide\node_modules\@theia\core\lib\electron-main\messaging\electron-messaging-contribution.js:116:20)
    at IpcMainImpl.<anonymous> (C:\Users\kittaakos\dev\arduino-ide\node_modules\@theia\core\lib\electron-main\messaging\electron-messaging-contribution.js:52:18)
    at IpcMainImpl.emit (node:events:394:28)
    at Object.<anonymous> (node:electron/js2c/browser_init:165:9959)
    at Object.emit (node:events:394:28)
    at Object.callbackTrampoline (node:internal/async_hooks:130:17)

eclipse-theia/theia#11435

  • Default save as path calculation is broken here. Steps: open an existing sketch (directories#user/alma), trigger save as, the default location must be directories#user and name new proposed name must be alma_copy_$date. But it's file:///c%3A/Users/kittaakos/Documents/Arduino/file%3A///c%253A/Users/kittaakos/Documents/Arduino/alma_copy_20220715154718. It's expected to be file:///c%3A/Users/kittaakos/Documents/Arduino/alma_copy_20220715154718

77ad13f

kittaakos pushed a commit that referenced this issue Aug 1, 2022
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
kittaakos pushed a commit that referenced this issue Aug 1, 2022
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
kittaakos pushed a commit that referenced this issue Aug 1, 2022
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
kittaakos pushed a commit that referenced this issue Aug 1, 2022
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants