Skip to content

Commit b7c680a

Browse files
Merge branch 'bugfix-12142' into release-6.6.4
2 parents fb247c9 + 9117412 commit b7c680a

2 files changed

Lines changed: 9 additions & 4 deletions

File tree

docs/notes/bugfix-12142.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
# mobileSensorReading("location", false) returns different data compared to mobileCurrentLocation() on Simulator

engine/src/mblsensor.cpp

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -204,7 +204,8 @@ void MCSensorGetLocationOfDevice(MCExecContext& ctxt, char *&r_location)
204204
if (MCSystemGetLocationReading(t_reading, false))
205205
{
206206
r_location = nil;
207-
MCCStringFormat(r_location, "%Lf,%Lf,%Lf", t_reading.latitude, t_reading.longitude, t_reading.altitude);
207+
// PM-2014-10-09: [[ Bug 12142 ]] The old %Lf format worked for device but failed on simulator.
208+
MCCStringFormat(r_location, "%lf,%lf,%lf", t_reading.latitude, t_reading.longitude, t_reading.altitude);
208209
}
209210
#endif /* MCSensorGetLocationOfDevice */
210211
}
@@ -256,7 +257,8 @@ void MCSensorGetHeadingOfDevice(MCExecContext& ctxt, char *&r_heading)
256257
if (MCSystemGetHeadingReading(t_reading, true))
257258
{
258259
r_heading = nil;
259-
MCCStringFormat(r_heading, "%Lf", t_reading.heading);
260+
// PM-2014-10-09: [[ Bug 12142 ]] The old %Lf format worked for device but failed on simulator. Currently, simulator does not support heading, acceleration or rotation, but since this might change in the future, use %lf instead
261+
MCCStringFormat(r_heading, "%lf", t_reading.heading);
260262
}
261263
#endif /* MCSensorGetHeadingOfDevice */
262264
}
@@ -296,7 +298,8 @@ void MCSensorGetAccelerationOfDevice(MCExecContext& ctxt, char *&r_acceleration)
296298
if (MCSystemGetAccelerationReading(t_reading, true))
297299
{
298300
r_acceleration = nil;
299-
MCCStringFormat(r_acceleration, "%Lf,%Lf,%Lf", t_reading.x, t_reading.y, t_reading.z);
301+
// PM-2014-10-09: [[ Bug 12142 ]] The old %Lf format worked for device but failed on simulator. Currently, simulator does not support heading, acceleration or rotation, but since this might change in the future, use %lf instead
302+
MCCStringFormat(r_acceleration, "%lf,%lf,%lf", t_reading.x, t_reading.y, t_reading.z);
300303
}
301304
#endif /* MCSensorGetAccelerationOfDevice */
302305
}
@@ -336,7 +339,8 @@ void MCSensorGetRotationRateOfDevice(MCExecContext& ctxt, char *&r_rotation_rate
336339
if (MCSystemGetRotationRateReading(t_reading, true))
337340
{
338341
r_rotation_rate = nil;
339-
MCCStringFormat(r_rotation_rate, "%Lf,%Lf,%Lf", t_reading.x, t_reading.y, t_reading.z);
342+
// PM-2014-10-09: [[ Bug 12142 ]] The old %Lf format worked for device but failed on simulator. Currently, simulator does not support heading, acceleration or rotation, but since this might change in the future, use %lf instead
343+
MCCStringFormat(r_rotation_rate, "%lf,%lf,%lf", t_reading.x, t_reading.y, t_reading.z);
340344
}
341345
#endif /* MCSensorGetRotationRateOfDevice */
342346
}

0 commit comments

Comments
 (0)