CopyQ icon indicating copy to clipboard operation
CopyQ copied to clipboard

Updated INI Values not Used When CopyQ Starts - Suspect Memory Corruption

Open dbareis opened this issue 3 years ago • 4 comments

Describe the bug I'm using "notification_bg" value as my "canary". If the INI works non-native notifications should display on black on yellow. When the INI fails it is white on black.

This didn't happen in the previous release I was using (v4.3 I think on Windows 10) but happens in 6.1.0 (at least for the same data/ini file).

Files for Test Case/Reproduction 1. MoreComplex-Fails-[get white on black]-copyq-DBAREIS-TOOLS-commands.zip 2. MoreComplex-Works-[Only added blank lines between sections]-copyq-DBAREIS-TOOLS-commands.zip

To Reproduce Steps to reproduce the behavior (on COPYQ 6.1.0):

  1. Stop CopyQ 6..1.0
  2. COPY the INI I have attached (1.MoreComplex-Fails-[get white on black]-copyq-DBAREIS-TOOLS-commands.zip)
  3. Start CopyQ
  4. Do something that causes a notification (invalid command etc)
  5. It will not have the correct colors
  6. Repeat the above with the INI from "1.MoreComplex-Works-[Only added blank lines between sections]-copyq-DBAREIS-TOOLS-commands.zip"
  7. It will now display with the correct colors

Note that I initially thought the INI handling needed these blank lines but I can make other non related data changes and get it to work which suggests some sort of in memory corruption of the INI object.

Expected behavior The values in the INI should always be used.

Version, OS and Environment (Get details from copyq version command if possible.)

  • 6.1.0, neither "....\copyq version" or "....\copyq copyq version" works
  • Windows 11

Tried for a Very Simple Text Case but that worked copyq-DBAREIS-TOOLS[very simple-works].zip

dbareis avatar May 18 '22 06:05 dbareis

I don't understand what is expected - only non-native notifications support changing appearance. The 2 ini files (1. and 2.) are using native notifications (specifically Windows native notification system). Only the "very simple" ini file uses the notification system from CopyQ.

Just to clarify: It is not possible to set color (and other attributes depending on the host system) in native notifications.

hluk avatar Jun 06 '22 15:06 hluk

I am only using non-native notifications and never saw native ones in my test. I will look at doing the tests again but in original (full) situation I definately had "native_notifications" as "false".

dbareis avatar Jun 06 '22 22:06 dbareis

BTW, replacing or changing the config file while the application is running is not supported.

hluk avatar Jun 07 '22 05:06 hluk

I do not replace it while running, I stop copyq, make changes and restart it...

dbareis avatar Jun 07 '22 05:06 dbareis