cc phabouzit (JIRA User)

Comment by Pierre Habouzit (JIRA)

{{
dispatchTimer.scheduleRepeating(deadline: DispatchTime.now(), interval: (1.0/60.0), leeway: DispatchTimeInterval.milliseconds(1))
}}

calls into

{{
public func scheduleRepeating(deadline: DispatchTime, interval: Double, leeway: DispatchTimeInterval = .nanoseconds(0)) {
__dispatch_source_set_timer(self as! DispatchSource, deadline.rawValue, UInt64(interval * Double(NSEC_PER_SEC)), UInt64(leeway.rawValue))
}
}}

which looks completely correct to me.

so investigating

Comment by Pierre Habouzit (JIRA)

When I trace in the debugger dispatch_source_set_timer() is called with the right interval value, but clearly the code runs at 6Hz and not 60

Comment by Pierre Habouzit (JIRA)

This is a project bug:

DispatchQueue.global(attributes: DispatchQueue.GlobalAttributes(rawValue: DispatchQueueAttributes.qosUserInteractive.rawValue))

is using rawValues that aren't compatible. GlobalAttributes are NOT DIspatchQueueAttributes.

DispatchQueue.global(attributes: [ .qosUserInteractive ]))

will fix it.

Comment by Daniel A. Steffen (JIRA)

I filed
<rdar://problem/27236887> DispatchQueue.GlobalAttributes is confusing
to track improving this aspect of the new API (the argument to global() doesn't need to be an OptionSet at all, or be called an attribute)

Comment by Brett Walker (JIRA)

Thanks for catching my mistake, Pierre. Mea culpa. I do agree with Daniel that the new dispatch timer API is quite a mouthful and could be improved. The use of rawValue here doesn't seem to be very Swifty as it encourages the kind of mistake I just made, instead of enforcing stricter type checking.