Bug 473792 - Unnecessary warning about lack of high-accuracy timing (for alarm times far into the future)
Summary: Unnecessary warning about lack of high-accuracy timing (for alarm times far i...
Status: RESOLVED FIXED
Alias: None
Product: kalarm
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: David Jarvie
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-08-27 08:37 UTC by Karl Ove Hufthammer
Modified: 2023-08-27 16:35 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In: 23.08.1


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Karl Ove Hufthammer 2023-08-27 08:37:10 UTC
SUMMARY
When adding an alarm, KAlarm shows the following warning:

    KAlarm does not provide high accuracy alarms.
    The alarm will trigger at the minute boundary before the specified time from now.

According to the changelog https://github.com/KDE/kalarm/blob/master/Changelog, the intention is to only show this warning if the alarm time would be triggered very soon (in the next few minutes?). But it‘s actually shown for alarms which only would be triggered far into the future, e.g., in a year’s time, where you don’t need high-accuracy timing.


STEPS TO REPRODUCE
1. Add a new alarm.
2. Set alarm time to a year into the future.
3. Click ‘OK’.

OBSERVED RESULT
Warning about lack of high-accuracy timing is shown.


EXPECTED RESULT
No warning should be shown. (It should be shown only for alarms set to be triggered in the x minutes, perhaps with x = 10.)

SOFTWARE/OS VERSIONS
Operating System: openSUSE Tumbleweed 20230822
KDE Plasma Version: 5.27.7
KDE Frameworks Version: 5.109.0
Qt Version: 5.15.10
Kernel Version: 6.4.11-1-default (64-bit)
Comment 1 David Jarvie 2023-08-27 16:35:49 UTC
You're correct in thinking that no warning should be shown for alarms >10 minutes ahead. That was the intention, but it was coded wrongly.
Fixed now for version 3.6.1 in KDE Gear 23.08.1, commit 65031e6e03c802a9e3669ee34c95e6d919cfff8e.

Thank you for reporting this.