Skip to content

Commit 4b232fa

Browse files
committed
Merge pull request #2581 from apple/stdlib-remove-deprecated-floating-point-apis
2 parents c2b8bb2 + 314dfe0 commit 4b232fa

File tree

5 files changed

+30
-39
lines changed

5 files changed

+30
-39
lines changed

stdlib/public/SDK/CoreGraphics/CGFloat.swift.gyb

Lines changed: 16 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -128,9 +128,9 @@ public struct CGFloat {
128128
return CGFloat(NativeType.signalingNaN)
129129
}
130130

131-
@available(*, deprecated, renamed: "nan")
131+
@available(*, unavailable, renamed: "nan")
132132
public static var quietNaN: CGFloat {
133-
return CGFloat(NativeType.nan)
133+
fatalError("unavailable")
134134
}
135135

136136
public static var greatestFiniteMagnitude: CGFloat {
@@ -261,9 +261,9 @@ public struct CGFloat {
261261
return native.isSignalingNaN
262262
}
263263

264-
@available(*, deprecated, renamed: "isSignalingNaN")
264+
@available(*, unavailable, renamed: "isSignalingNaN")
265265
public var isSignaling: Bool {
266-
return native.isSignalingNaN
266+
fatalError("unavailable")
267267
}
268268

269269
public var isCanonical: Bool {
@@ -294,33 +294,25 @@ public struct CGFloat {
294294
}
295295

296296
extension CGFloat {
297-
@available(*, deprecated, message: "use CGFloat.leastNormalMagnitude")
298-
@_transparent public static var min: CGFloat {
299-
#if arch(i386) || arch(arm)
300-
return CGFloat(FLT_MIN)
301-
#else
302-
return CGFloat(DBL_MIN)
303-
#endif
297+
@available(*, unavailable, renamed: "leastNormalMagnitude")
298+
public static var min: CGFloat {
299+
fatalError("unavailable")
304300
}
305301

306-
@available(*, deprecated, message: "use CGFloat.greatestFiniteMagnitude")
307-
@_transparent public static var max: CGFloat {
308-
#if arch(i386) || arch(arm)
309-
return CGFloat(FLT_MAX)
310-
#else
311-
return CGFloat(DBL_MAX)
312-
#endif
302+
@available(*, unavailable, renamed: "greatestFiniteMagnitude")
303+
public static var max: CGFloat {
304+
fatalError("unavailable")
313305
}
314306
}
315307

316-
@available(*, unavailable, message: "use CGFloat.leastNormalMagnitude")
308+
@available(*, unavailable, renamed: "CGFloat.leastNormalMagnitude")
317309
public var CGFLOAT_MIN: CGFloat {
318-
fatalError("can't retrieve unavailable property")
310+
fatalError("unavailable")
319311
}
320312

321-
@available(*, unavailable, message: "use CGFloat.greatestFiniteMagnitude")
313+
@available(*, unavailable, renamed: "CGFloat.greatestFiniteMagnitude")
322314
public var CGFLOAT_MAX: CGFloat {
323-
fatalError("can't retrieve unavailable property")
315+
fatalError("unavailable")
324316
}
325317

326318
extension CGFloat : CustomReflectable {
@@ -753,9 +745,9 @@ public func fmin(_ lhs: CGFloat, _ rhs: CGFloat) -> CGFloat {
753745

754746
@_transparent
755747
@warn_unused_result
756-
@available(*, deprecated, message: "use the floatingPointClass property.")
748+
@available(*, unavailable, message: "use the floatingPointClass property.")
757749
public func fpclassify(_ x: CGFloat) -> Int {
758-
return fpclassify(x.native)
750+
fatalError("unavailable")
759751
}
760752

761753
@available(*, unavailable, message: "use the isNormal property.")

stdlib/public/core/FloatingPoint.swift.gyb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -813,7 +813,7 @@ extension BinaryFloatingPoint {
813813
return true
814814
}
815815

816-
@available(*, deprecated, renamed: "isSignalingNaN")
816+
@available(*, unavailable, renamed: "isSignalingNaN")
817817
public var isSignaling: Bool {
818818
return isSignalingNaN
819819
}

stdlib/public/core/FloatingPointTypes.swift.gyb

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -199,8 +199,8 @@ extension ${Self}: BinaryFloatingPoint {
199199
return FloatingPointSign(rawValue: Int(bitPattern >> ${RawSignificand}(shift)))!
200200
}
201201

202-
@available(*, deprecated, message: "Use the .sign property instead.")
203-
public var isSignMinus: Bool { return sign == .minus }
202+
@available(*, unavailable, renamed: "sign")
203+
public var isSignMinus: Bool { Builtin.unreachable() }
204204

205205
public var exponentBitPattern: UInt {
206206
return UInt(bitPattern >> UInt${bits}(${Self}.significandBitCount)) &
@@ -329,8 +329,8 @@ extension ${Self}: BinaryFloatingPoint {
329329
return ${Self}(nan: 0, signaling: true)
330330
}
331331

332-
@available(*, deprecated, renamed: "nan")
333-
public static var quietNaN: ${Self} { return nan }
332+
@available(*, unavailable, renamed: "nan")
333+
public static var quietNaN: ${Self} { Builtin.unreachable()}
334334

335335
public static var greatestFiniteMagnitude: ${Self} {
336336
return ${Self}(sign: .plus,

test/1_stdlib/Float.swift

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -167,28 +167,28 @@ func checkInf(_ inf: TestFloat) {
167167
_precondition(!inf.isSubnormal)
168168
_precondition(inf.isInfinite)
169169
_precondition(!inf.isNaN)
170-
_precondition(!inf.isSignaling)
170+
_precondition(!inf.isSignalingNaN)
171171
}
172172

173173
func testInf() {
174174
var stdlibPlusInf = TestFloat.infinity
175175
checkInf(stdlibPlusInf)
176-
_precondition(!stdlibPlusInf.isSignMinus)
176+
_precondition(stdlibPlusInf.sign == .plus)
177177
_precondition(stdlibPlusInf.floatingPointClass == .positiveInfinity)
178178

179179
var stdlibMinusInf = -TestFloat.infinity
180180
checkInf(stdlibMinusInf)
181-
_precondition(stdlibMinusInf.isSignMinus)
181+
_precondition(stdlibMinusInf.sign == .minus)
182182
_precondition(stdlibMinusInf.floatingPointClass == .negativeInfinity)
183183

184184
var computedPlusInf = 1.0 / noinlinePlusZero()
185185
checkInf(computedPlusInf)
186-
_precondition(!computedPlusInf.isSignMinus)
186+
_precondition(computedPlusInf.sign == .plus)
187187
_precondition(computedPlusInf.floatingPointClass == .positiveInfinity)
188188

189189
var computedMinusInf = -1.0 / noinlinePlusZero()
190190
checkInf(computedMinusInf)
191-
_precondition(computedMinusInf.isSignMinus)
191+
_precondition(computedMinusInf.sign == .minus)
192192
_precondition(computedMinusInf.floatingPointClass == .negativeInfinity)
193193

194194
_precondition(stdlibPlusInf == computedPlusInf)
@@ -207,7 +207,7 @@ testInf()
207207
//===---
208208

209209
func checkNaN(_ nan: TestFloat) {
210-
_precondition(!nan.isSignMinus)
210+
_precondition(nan.sign == .plus)
211211
_precondition(!nan.isNormal)
212212
_precondition(!nan.isFinite)
213213
_precondition(!nan.isZero)
@@ -218,15 +218,15 @@ func checkNaN(_ nan: TestFloat) {
218218

219219
func checkQNaN(_ qnan: TestFloat) {
220220
checkNaN(qnan)
221-
_precondition(!qnan.isSignaling)
221+
_precondition(!qnan.isSignalingNaN)
222222
_precondition(qnan.floatingPointClass == .quietNaN)
223223
}
224224

225225
func checkSNaN(_ snan: TestFloat) {
226226
checkNaN(snan)
227227
// sNaN cannot be fully supported on i386.
228228
#if !arch(i386)
229-
_precondition(snan.isSignaling)
229+
_precondition(snan.isSignalingNaN)
230230
_precondition(snan.floatingPointClass == .signalingNaN)
231231
#endif
232232
}
@@ -235,7 +235,7 @@ func testNaN() {
235235
var stdlibDefaultNaN = TestFloat.nan
236236
checkQNaN(stdlibDefaultNaN)
237237

238-
var stdlibQNaN = TestFloat.quietNaN
238+
var stdlibQNaN = TestFloat.nan
239239
checkQNaN(stdlibQNaN)
240240

241241
var stdlibSNaN = TestFloat.signalingNaN

test/IDE/complete_literal.swift

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@
1717
1.1.#^LITERAL2^#
1818
}
1919
// LITERAL2: Begin completions
20-
// LITERAL2-DAG: Decl[InstanceVar]/CurrNominal: isSignMinus[#Bool#]; name=isSignMinus{{$}}
2120
// LITERAL2-DAG: Decl[InstanceVar]/CurrNominal: isNormal[#Bool#]; name=isNormal{{$}}
2221
// LITERAL2-DAG: Decl[InstanceVar]/CurrNominal: isFinite[#Bool#]; name=isFinite{{$}}
2322
// LITERAL2-DAG: Decl[InstanceVar]/CurrNominal: isZero[#Bool#]; name=isZero{{$}}

0 commit comments

Comments
 (0)