[mythtv-commits] Ticket #11197: Fix event times in MythZoneMinder broken by the UTC changes

[MythTV]

#11197: Fix event times in MythZoneMinder broken by the UTC changes
-------------------------------------+-----------------------------
 Reporter:  mythtv@…                 |          Owner:  danielk
     Type:  Patch - Bug Fix          |         Status:  accepted
 Priority:  minor                    |      Milestone:  0.26.1
Component:  Plugin - MythZoneminder  |        Version:  Master Head
 Severity:  medium                   |     Resolution:
 Keywords:                           |  Ticket locked:  0
-------------------------------------+-----------------------------

Comment (by danielk):

 paulh, the standard is now for all dates to be UTC unless there is a good
 reason for them not to be. This means we don't need to be checking what
 timezone a date time is in, if the variable name contains "localTime" it
 is local time, otherwise it is not. If there is a real performance problem
 we can of course rename the variable and use local time internally. Is it
 20k event per second, per hour, per day?

 BTW We're ironically doing a lot fewer utc<->localtime conversions overall
 now. We used to be doing a few hundred conversions per second in the
 frontend because QDateTime::currentDateTime() uses it. Now with
 QDateTime::currentDateTimeUtc() we avoid converting to local time most of
 the time and it has a measurable positive effect on both CPU usage and
 responsiveness.

-- 
Ticket URL: <http://code.mythtv.org/trac/ticket/11197#comment:5>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center