Project

General

Profile

Bug #793

alarm plugin needs rewrite

Added by Joe Kamprad over 6 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Major
Assignee:
-
Category:
plugins/alarm
Target version:
Start date:
May 06, 2018
Due date:
% Done:

100%

Estimated time:
Affects version:

Description

When i enable the alarm plugin it force to enable an alarm, set to the time i do enable the plugin.
This can be very annoying if you do not remember this (or first time users).
Alarm should be not be enabled without active setting it.

Bildschirmfoto vom 2018-05-06 16-39-40.png (49.6 KB) Bildschirmfoto vom 2018-05-06 16-39-40.png screenshot alarm settings Joe Kamprad, May 06, 2018 14:40

History

#1 Updated by John Lindgren over 6 years ago

The current alarm plugin has several serious problems:
- Unsafe use of pthread_cancel() -- on a thread calling into GTK!
- Heavy use of GTK, which we are moving away from in favor of Qt
- Usability issues such as this one

In a future version we'll probably just remove it. A replacement shouldn't be terribly hard to write.

#2 Updated by John Lindgren over 6 years ago

  • Category set to plugins/alarm
  • Subject changed from alarm plugin (audicious 3.9) enables alarm automatic after enable the lugin to alarm plugin needs rewrite

#3 Updated by John Lindgren over 6 years ago

  • Priority changed from Trivial to Major

#4 Updated by John Lindgren about 2 years ago

  • % Done changed from 0 to 100
  • Target version set to 4.3
  • Status changed from New to Closed

It's been over 4 years and nothing has been done to save the alarm plugin. I've removed it:
https://github.com/audacious-media-player/audacious-plugins/commit/28d18e44498325b237520c850a555841b2770fea

Also available in: Atom PDF