0003158: Зависание при изменении масштаба с 9 на 10 или с 10 на 9 при разрешении 3840х2160 - SASGIS

SASGIS - SAS.Планета

View Issue Details

ID

Project

Category

View Status

Date Submitted

Last Update

0003158

SAS.Планета

[All Projects] Баг

public

06-12-2016 01:06

23-10-2017 20:05

Reporter

Gromov

Assigned To

vdemidov

Priority

normal

Severity

block

Reproducibility

always

Status

resolved

Resolution

fixed

Platform

OS

Windows

OS Version

10, 64-bit

Product Version

160707

Target Version

181221

Fixed in Version

181221

Summary

0003158: Зависание при изменении масштаба с 9 на 10 или с 10 на 9 при разрешении 3840х2160

Description

Программа зависает, если в системе выставлено разрешение 3840х2160, в свойствах приложения в Windows во вкладке "Совместимость" стоит галочка "Отключить масштабирование изображения при высоком разрешении" и масштаб карты в программе меняется с 9-го на 10-й или наоборот. В случае, если не включать эту опцию, изображение выглядит размытым.

Steps To Reproduce

1. Установить или распаковать программу
2. Выбрать исполняемый файл и в контекстном меню выбрать пункт "Свойства"
3. Во вкладке "Совместимость" поставить галочку "Отключить масштабирование изображения при высоком разрешении"
4. Приблизить карту до масштаба 9, попробовать перейти к масштабу 10 - программа зависает, отображая карту масштаба 9.

Additional Information

Tags

No tags attached.

Relationships

has duplicate

0003291

closed

Locking up

Attached Files

SASPlanet.Debug.elf (92,484) 06-12-2016 23:37
http://www.sasgis.org/mantis/file_download.php?file_id=2126&type=bug

2017.01.23_SASPlanet.Debug.elf (91,154) 26-01-2017 22:34
http://www.sasgis.org/mantis/file_download.php?file_id=2141&type=bug

Issue History

Date Modified

Username

Field

Change

06-12-2016 01:06

New Issue

06-12-2016 12:08

Project

Домен, сайт, форум, багтрекер => SAS.Планета

06-12-2016 12:12

Note Added: 0017712

06-12-2016 12:12

Status

new => feedback

06-12-2016 12:30

Note Added: 0017713

06-12-2016 12:30

Status

feedback => new

06-12-2016 12:32

Note Added: 0017714

06-12-2016 23:37

File Added: SASPlanet.Debug.elf

06-12-2016 23:39

Note Added: 0017716

07-12-2016 06:19

Note Added: 0017717

07-12-2016 07:30

Note Added: 0017718

23-01-2017 01:13

Note Added: 0017782

23-01-2017 05:50

Note Added: 0017783

26-01-2017 22:34

File Added: 2017.01.23_SASPlanet.Debug.elf

26-01-2017 22:34

Note Added: 0017784

27-01-2017 07:25

Note Added: 0017785

27-01-2017 07:29

Note Added: 0017786

27-01-2017 07:39

Note Edited: 0017786

bug_revision_view_page.php?bugnote_id=17786#r7124

28-01-2017 16:01

Note Added: 0017788

28-01-2017 16:03

Status

new => confirmed

28-01-2017 16:03

Product Version

=> 160707

28-01-2017 16:03

Target Version

=> 26xxxx

28-01-2017 16:03

Summary

зависание при изменении масштаба с 9 на 10 или с 10 на 9 при разрешении 3840х2160 => Зависание при изменении масштаба с 9 на 10 или с 10 на 9 при разрешении 3840х2160

28-01-2017 16:58

Note Added: 0017789

28-01-2017 16:58

Assigned To

=> vdemidov

28-01-2017 16:58

Status

confirmed => assigned

28-01-2017 17:01

Note Added: 0017790

28-01-2017 17:01

Assigned To

vdemidov =>

28-01-2017 17:01

Status

assigned => feedback

30-01-2017 01:10

Note Added: 0017791

30-01-2017 01:10

Status

feedback => new

30-01-2017 08:22

Status

new => resolved

30-01-2017 08:22

Fixed in Version

=> 181221

30-01-2017 08:22

Resolution

open => fixed

30-01-2017 08:22

Assigned To

=> vdemidov

30-01-2017 08:23

Target Version

26xxxx => 181221

23-10-2017 20:05

Relationship added

has duplicate 0003291

Notes

Я, кажется, понял в чем проблема. Классический дедлок связанный с рекурсивным вызовом примитива синхронизации не поддерживающего рекурсивных вызовов. Да еще и при редком стечении обстоятельств. Проблема в TViewPortState. В том что у него есть и блокировка FCS, и FView со своим вызовом уведомлений подписчиков под этой блокировкой.

Как всегда исправить можно кучей разных способов:

Можно сделать FCS поддерживающим рекурсивные вызовы. Это несколько смягчит, но, как мне кажется, не исправит проблему полностью, потому что в свое время я не рассчитывал TViewPortState на повторные вхождения и может вылезти что-то еще. А может и не вылезти.

Можно вынести изменение FView из под блокировки. Но тогда может возникнуть хитрая ситуация гонки, если два потока вызвали изменение TViewPortState и действия одного из них просто потерялись, потому что он пользовался еще не обновленным значением FView.

Можно поковырять ILocalCoordConverterChangeableInternal и сделать, что бы обязанностью вызывать нотификаторы подписчиков было у изменяющего. То есть SetConverter изменяет значение, но не дергает нотификаторы, а только возвращает bool указывающий на необходимость этого, а уже TViewPortState дергает добавленный метод DoNotify после освобождения блокировки.