[drape] Fixed OnEnterBackground notification. #3399

rtsisyk · 2022-09-13T13:06:35Z

vng commented

2022-09-13 13:06:35 +00:00

(Migrated from github.com)

Also fixed grey position arrow, if user has invalid time settings.

I can't reproduce, but also should fix useless "Continue detecting your current location" in iOS.

Also fixed grey position arrow, if user has invalid time settings. I can't reproduce, but also should fix useless "Continue detecting your current location" in iOS.

rtsisyk reviewed 2022-09-13 13:06:35 +00:00

biodranik (Migrated from github.com) reviewed 2022-09-13 13:38:02 +00:00

biodranik (Migrated from github.com) left a comment

Фикс правильный, но довольно рискованный, в плане, что могут отвалиться какие-то старые костыли. Надо потестить на девайсах.

base/timer.cpp

					
				@ -22,6 +22,8 @@ namespace base

				// static

				double Timer::LocalTime()

				{

				  /// @todo duration_cast<seconds>(system_clock::now().time_since_epoch()).count() ?

biodranik (Migrated from github.com) commented

2022-09-13 13:33:29 +00:00

Хорошая идея. Но надо протестить точность этой функции на вёдрах и айфонах.
По коду глянуть, точно ли нам нужно в таком виде LocalTime. Может надо что-то другое, чтобы избежать багов, когда юзер меняет локальное время.
Этот статический метод вообще можно вынести в отдельный хидер, и не требовать линковки прямо со всей base либой.

1. Хорошая идея. Но надо протестить точность этой функции на вёдрах и айфонах. 2. По коду глянуть, точно ли нам нужно в таком виде LocalTime. Может надо что-то другое, чтобы избежать багов, когда юзер меняет локальное время. 3. Этот статический метод вообще можно вынести в отдельный хидер, и не требовать линковки прямо со всей base либой.

biodranik (Migrated from github.com) reviewed 2022-09-13 21:34:42 +00:00

drape_frontend/frontend_renderer.cpp

biodranik (Migrated from github.com) commented

2022-09-13 21:34:42 +00:00

Я правильно понимаю, что много переменных в этом методе теперь может модифицироваться из разных потоков? И соответственно к рандомным проблемам из-за этого?