UpdateEx

Здесь можно обсудить ваш любимый плагин.
Shmuel
Posts: 5968
Joined: Thu 23 Mar, 2006 21:36
Location: Israel
Has thanked: 24 times
Been thanked: 257 times

Re: Глюки и баги, появившиеся в «ночных сборках»

Post by Shmuel »

Latest post of the previous page:

Новая информация по падению при выходе из Фара в 3539:

Необходимое (и похоже, достаточное) условие для падения - наличие плагина UpdateEx. Во всяком случае:

а) без него ни разу не удалось воспроизвести падение
б) удаётся воспроизвести, когда он - единственный установленный плагин (падает примерно 1 раз на 10 запусков Фара; и, как уже говорилось, не падает в релизе Фара, собранном VC++ 2010).

Посмотрел в код плагина, там что-то с потоками (threads), но дальше разбираться не стал.
Last edited by Shmuel on Thu 01 Jan, 1970 01:00, edited 0 times in total.
Reason: Добавлена ссылка на исходную тему

User avatar
Alexey Samlyukov
Posts: 1760
Joined: Tue 15 Mar, 2005 17:28
Been thanked: 5 times
Contact:

Re: Глюки и баги, появившиеся в «ночных сборках»

Post by Alexey Samlyukov »

интересны подробности. можно в личку :)

User avatar
Alexey Samlyukov
Posts: 1760
Joined: Tue 15 Mar, 2005 17:28
Been thanked: 5 times
Contact:

Re: Глюки и баги, появившиеся в «ночных сборках»

Post by Alexey Samlyukov »

Shmuel
в общем, очерадная пятиминутка входов-выходов на присланном тобой архиве дает под отладчиком:
Поток 'Поток Win32' (0x174c) завершился с кодом 0 (0x0).
Программа "[5480] Far.exe: Машинный код" завершилась с кодом 0 (0x0).
ошибка мной не выявлена, сорри...

Shmuel
Posts: 5968
Joined: Thu 23 Mar, 2006 21:36
Location: Israel
Has thanked: 24 times
Been thanked: 257 times

Re: Глюки и баги, появившиеся в «ночных сборках»

Post by Shmuel »

Alexey Samlyukov wrote:ошибка мной не выявлена, сорри...
Если в файле UpdateEx.cpp заменить в строке 2457 Sleep(1000) на Sleep(100), то дебаг-сборка Фара VC++ 2010 падает на выходе 10 раз из 10. Та же картина и с релизом GCC. Сорри.

User avatar
HaRT
Moderator
Posts: 9238
Joined: Tue 30 Aug, 2005 17:21
Location: Санкт-Петербург
Has thanked: 93 times
Been thanked: 197 times

Re: Глюки и баги, появившиеся в «ночных сборках»

Post by HaRT »

Shmuel wrote:
Alexey Samlyukov wrote:ошибка мной не выявлена, сорри...
Если в файле UpdateEx.cpp заменить в строке 2457 Sleep(1000) на Sleep(100), то дебаг-сборка Фара VC++ 2010 падает на выходе 10 раз из 10. Сорри.
Шах и мат! :ROFL:
Фар есть инструмент, а не нянька. © 2009 DrKnS

User avatar
Alexey Samlyukov
Posts: 1760
Joined: Tue 15 Mar, 2005 17:28
Been thanked: 5 times
Contact:

Re: UpdateEx

Post by Alexey Samlyukov »

нужна помощь, тут лежит дебаг версия фара и плагин с правкой при которой должно падать сто процентов:
Shmuel wrote:Если в файле UpdateEx.cpp заменить в строке 2457 Sleep(1000) на Sleep(100), то дебаг-сборка Фара VC++ 2010 падает на выходе 10 раз из 10.
просьба уронить, т.к. у меня не получается, и желательно так, чтоб я понял, что же там криво-то :)
http://dl.dropboxusercontent.com/u/6567 ... CC-7.1.rar
или
http://dl.dropboxusercontent.com/u/6567 ... x86.vc.rar
Last edited by Alexey Samlyukov on Thu 01 Jan, 1970 01:00, edited 0 times in total.
Reason: Добавлена ссылка на цитируемое сообщение

User avatar
Alexey Samlyukov
Posts: 1760
Joined: Tue 15 Mar, 2005 17:28
Been thanked: 5 times
Contact:

Re: Глюки и баги, появившиеся в «ночных сборках»

Post by Alexey Samlyukov »

HaRT ты повторил? научи. или что это за зубоскаленье?

User avatar
HaRT
Moderator
Posts: 9238
Joined: Tue 30 Aug, 2005 17:21
Location: Санкт-Петербург
Has thanked: 93 times
Been thanked: 197 times

Re: Глюки и баги, появившиеся в «ночных сборках»

Post by HaRT »

Alexey Samlyukov wrote:или что это за зубоскаленье?
Именно зубоскаленье.
Если крэш даже хотя бы на одной машине появляется/исчезает в зависимости от параметра Sleep, мне сразу очевидно, что межпоточное взаимодействие запрограммировано… ээ… недостаточно качественно. В таких случаях воспроизведение падения на компе разработчика редко помогает устранить проблему. Просто надо делать честную синхронизацию потоков.
Фар есть инструмент, а не нянька. © 2009 DrKnS

Shmuel
Posts: 5968
Joined: Thu 23 Mar, 2006 21:36
Location: Israel
Has thanked: 24 times
Been thanked: 257 times

Re: UpdateEx

Post by Shmuel »

Алексей, успокойся, никто не пытается тебя "достать".

Проявление проблемы, очевидно, зависит от сочетания времени Sleep(X) в вышеупомянутом месте плагина и времени выхода самого Фара. Разница в моих и твоих опытах, скорее всего, вызвана тем, что ты пробовал на чистом профиле, а я, хоть и без плагинов, но все базы убранных плагинов лежали в профиле (то-есть, по-видимому, при этом время выхода Фара больше).

И, кстати, при том, что Фар падает у меня 10 раз из 10 в отладчике VC++ 2010 Express, при запуске этого же Фара с командной строки падения не происходит (или оно визуально не проявляется).

Очевидно, что проблема либо в плагине, либо в Фаре. Мне кажется - первое, хотя 100-процентной уверенности нет.

User avatar
Alexey Samlyukov
Posts: 1760
Joined: Tue 15 Mar, 2005 17:28
Been thanked: 5 times
Contact:

Re: UpdateEx

Post by Alexey Samlyukov »

Shmuel
просто для проверки, если перестанет падать, будем перелопачивать плаг...
http://dl.dropboxusercontent.com/u/6567 ... 6_test.rar

Shmuel
Posts: 5968
Joined: Thu 23 Mar, 2006 21:36
Location: Israel
Has thanked: 24 times
Been thanked: 257 times

Re: UpdateEx

Post by Shmuel »

Alexey Samlyukov wrote:просто для проверки, если перестанет падать, будем перелопачивать плаг...
http://dl.dropboxusercontent.com/u/6567 ... 6_test.rar
Вроде не падает (сделал по 20 запусков в VC++ 2010 и GCC).

User avatar
Alexey Samlyukov
Posts: 1760
Joined: Tue 15 Mar, 2005 17:28
Been thanked: 5 times
Contact:

Re: UpdateEx

Post by Alexey Samlyukov »

спасибо. сделан пока грязный фикс. надо хорошо покумекать как переделать.

Shmuel
Posts: 5968
Joined: Thu 23 Mar, 2006 21:36
Location: Israel
Has thanked: 24 times
Been thanked: 257 times

Re: UpdateEx

Post by Shmuel »

Alexey Samlyukov wrote:спасибо. сделан пока грязный фикс. надо хорошо покумекать как переделать.
Хотел бы добавить, что проверка на моей (или другой одной) системе была бы информативной в случае продолжения падений, но то, что они исчезли - далеко не гарантия правильности фикса. Как сказал HaRT выше: "просто надо делать честную синхронизацию потоков".

User avatar
Alexey Samlyukov
Posts: 1760
Joined: Tue 15 Mar, 2005 17:28
Been thanked: 5 times
Contact:

Re: UpdateEx

Post by Alexey Samlyukov »

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

User avatar
DrKnS
Posts: 5484
Joined: Thu 04 Aug, 2005 06:44
Location: Kyiv
Has thanked: 6 times
Been thanked: 272 times

Re: UpdateEx

Post by DrKnS »

Alexey Samlyukov wrote:там не столько в синхре дело, там просто криво реализован поток, вернее, в оригинале его реализация работала, а после навешивания разных фишек уже не тянет.
Вот не надо, в оригинале всё честно синхронизировалось.

User avatar
Alexey Samlyukov
Posts: 1760
Joined: Tue 15 Mar, 2005 17:28
Been thanked: 5 times
Contact:

Re: UpdateEx

Post by Alexey Samlyukov »

так и сказал, что не так?

Post Reply

Return to “Обсуждение плагинов”