[android] Fix download progress #1119

rtsisyk · 2021-08-18T09:32:38Z

rtsisyk commented

2021-08-18 09:32:38 +00:00

mTrackingIds (Java) and g_completionHandlers (C++) do the same job.

Signed-off-by: Roman Tsisyk roman@tsisyk.com

mTrackingIds (Java) and g_completionHandlers (C++) do the same job. Signed-off-by: Roman Tsisyk <roman@tsisyk.com>

biodranik (Migrated from github.com) approved these changes 2021-08-18 09:45:26 +00:00

biodranik (Migrated from github.com) left a comment

А где next для курсора?

android/src/com/mapswithme/maps/downloader/MapDownloadManager.java

					
				@ -147,4 +147,3 @@

				    mProgressTracker.remove(requestId);

				    mDownloadManager.remove(requestId);

				  }

biodranik (Migrated from github.com) commented

2021-08-18 09:40:50 +00:00

Имеет смысл оставлять метод с одной строчкой, или лучше удалить и упростить?

android/src/com/mapswithme/maps/downloader/MapDownloadManager.java

					
				@ -153,4 +152,3 @@

				    mProgressTracker.remove(id);

				    MapManager.nativeOnDownloadFinished(status, id);

				  }

biodranik (Migrated from github.com) commented

2021-08-18 09:41:09 +00:00

Имеет смысл оставлять метод с одной строчкой, или лучше удалить и упростить?

android/src/com/mapswithme/maps/downloader/MapDownloadProgressTracker.java

biodranik (Migrated from github.com) commented

2021-08-18 09:41:40 +00:00

Т.е. пока опрос будет лупить постоянно, как и раньше?

android/src/com/mapswithme/maps/downloader/MapDownloadProgressTracker.java

biodranik (Migrated from github.com) commented

2021-08-18 09:44:20 +00:00

А start() заново вызовется потом при запуске новой загрузки? В true значение потом переключится?

rtsisyk reviewed 2021-08-18 09:46:20 +00:00

android/src/com/mapswithme/maps/downloader/MapDownloadManager.java

					
				@ -153,4 +152,3 @@

				    mProgressTracker.remove(id);

				    MapManager.nativeOnDownloadFinished(status, id);

				  }

rtsisyk commented

2021-08-18 09:46:20 +00:00

Там имеет смысл всё упростить. Бесконечные прослойки.

rtsisyk reviewed 2021-08-18 09:47:01 +00:00

android/src/com/mapswithme/maps/downloader/MapDownloadProgressTracker.java

rtsisyk commented

2021-08-18 09:47:01 +00:00

Здесь запускаю отложенно, потому что до возврата из метода enquee() в g_completionHandlers все еще пусто.

vng (Migrated from github.com) reviewed 2021-08-18 09:47:50 +00:00

android/src/com/mapswithme/maps/downloader/MapDownloadProgressTracker.java

vng (Migrated from github.com) commented

2021-08-18 09:47:50 +00:00

В таком случае мы будем вызывать MapManager.nativeOnDownloadProgress, после MapManager.nativeOnDownloadFinished.

Я хз какой там код внутри С++, но ты проверял что тут будет все ок? Для закончившейся закачки и для закачки с ошибкой?

В таком случае мы будем вызывать MapManager.nativeOnDownloadProgress, после MapManager.nativeOnDownloadFinished. Я хз какой там код внутри С++, но ты проверял что тут будет все ок? Для закончившейся закачки и для закачки с ошибкой?

rtsisyk reviewed 2021-08-18 09:47:59 +00:00

android/src/com/mapswithme/maps/downloader/MapDownloadProgressTracker.java

rtsisyk commented

2021-08-18 09:47:59 +00:00

organicmaps/organicmaps#1119/files вот здесь должно запускаться каждый раз при добавлении нового файла.

https://git.omaps.dev/organicmaps/organicmaps/pulls/1119/files#diff-83366359a6d47e881b1196ea2a11a50c82fe4f212a5df81953dafc35272e11a0L139 вот здесь должно запускаться каждый раз при добавлении нового файла.