search

aromajoin / material-showcase-ios

✨ An elegant way to guide your beloved users in iOS apps - Material Showcase.
https://aromajoin.com/
Other
362 stars 128 forks source link

Result of call to 'insetBy(dx:dy:)' is unused #117

Open zeusent opened 4 years ago

zeusent commented 4 years ago

Xcode is throwing a warning on this line: https://github.com/aromajoin/material-showcase-ios/blob/49504ddfedbf9ab5d089504303ef4edefd9a7372/MaterialShowcase/MaterialShowcase%2BCalculations.swift#L43. Looking at it and the line above I think they should be changed to this:

var expandedBounds = CGRect(x: targetCenterX, y: targetCenterY, width: 0, height: 0)
expandedBounds = expandedBounds.insetBy(dx: -expandedRadius, dy: -expandedRadius);