UpdateEx — обновление Фара и плагинов с плагринга

Здесь можно обсудить ваш любимый плагин.
User avatar
Alexey Samlyukov
Posts: 1765
Joined: Tue 15 Mar, 2005 17:28
Been thanked: 7 times
Contact:

Re: UpdateEx

Post by Alexey Samlyukov »

Latest post of the previous page:

нужна помощь, тут лежит дебаг версия фара и плагин с правкой при которой должно падать сто процентов:
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: 1765
Joined: Tue 15 Mar, 2005 17:28
Been thanked: 7 times
Contact:

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

Post by Alexey Samlyukov »

HaRT ты повторил? научи. или что это за зубоскаленье?
User avatar
HaRT
Moderator
Posts: 10863
Joined: Tue 30 Aug, 2005 17:21
Has thanked: 221 times
Been thanked: 358 times

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

Post by HaRT »

Alexey Samlyukov wrote:или что это за зубоскаленье?
Именно зубоскаленье.
Если крэш даже хотя бы на одной машине появляется/исчезает в зависимости от параметра Sleep, мне сразу очевидно, что межпоточное взаимодействие запрограммировано… ээ… недостаточно качественно. В таких случаях воспроизведение падения на компе разработчика редко помогает устранить проблему. Просто надо делать честную синхронизацию потоков.
Фар есть инструмент, а не нянька. © 2009 DrKnS
Shmuel
Posts: 6834
Joined: Thu 23 Mar, 2006 21:36
Location: Israel
Has thanked: 41 times
Been thanked: 530 times

Re: UpdateEx

Post by Shmuel »

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

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

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

Очевидно, что проблема либо в плагине, либо в Фаре. Мне кажется - первое, хотя 100-процентной уверенности нет.
User avatar
Alexey Samlyukov
Posts: 1765
Joined: Tue 15 Mar, 2005 17:28
Been thanked: 7 times
Contact:

Re: UpdateEx

Post by Alexey Samlyukov »

Shmuel
просто для проверки, если перестанет падать, будем перелопачивать плаг...
http://dl.dropboxusercontent.com/u/6567 ... 6_test.rar
Shmuel
Posts: 6834
Joined: Thu 23 Mar, 2006 21:36
Location: Israel
Has thanked: 41 times
Been thanked: 530 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: 1765
Joined: Tue 15 Mar, 2005 17:28
Been thanked: 7 times
Contact:

Re: UpdateEx

Post by Alexey Samlyukov »

спасибо. сделан пока грязный фикс. надо хорошо покумекать как переделать.
Shmuel
Posts: 6834
Joined: Thu 23 Mar, 2006 21:36
Location: Israel
Has thanked: 41 times
Been thanked: 530 times

Re: UpdateEx

Post by Shmuel »

Alexey Samlyukov wrote:спасибо. сделан пока грязный фикс. надо хорошо покумекать как переделать.
Хотел бы добавить, что проверка на моей (или другой одной) системе была бы информативной в случае продолжения падений, но то, что они исчезли - далеко не гарантия правильности фикса. Как сказал HaRT выше: "просто надо делать честную синхронизацию потоков".
User avatar
Alexey Samlyukov
Posts: 1765
Joined: Tue 15 Mar, 2005 17:28
Been thanked: 7 times
Contact:

Re: UpdateEx

Post by Alexey Samlyukov »

там не столько в синхре дело, там просто криво реализован поток, вернее, в оригинале его реализация работала, а после навешивания разных фишек уже не тянет. будет переписан, но позже. сейчас не до плагина.
User avatar
DrKnS
Posts: 6114
Joined: Thu 04 Aug, 2005 06:44
Location: Kyiv
Has thanked: 12 times
Been thanked: 426 times

Re: UpdateEx

Post by DrKnS »

Alexey Samlyukov wrote:там не столько в синхре дело, там просто криво реализован поток, вернее, в оригинале его реализация работала, а после навешивания разных фишек уже не тянет.
Вот не надо, в оригинале всё честно синхронизировалось.
User avatar
Alexey Samlyukov
Posts: 1765
Joined: Tue 15 Mar, 2005 17:28
Been thanked: 7 times
Contact:

Re: UpdateEx

Post by Alexey Samlyukov »

так и сказал, что не так?
User avatar
John Doe
Бюрократ
Posts: 13853
Joined: Wed 27 Apr, 2005 20:42
Has thanked: 73 times
Been thanked: 428 times
Contact:

Re: UpdateEx

Post by John Doe »

Выбираю плюсиком PictureView для обновления, но в результате почему-то устанавливается анси-версия (Unpacking PictureView3A.7z)
User avatar
Alexey Samlyukov
Posts: 1765
Joined: Tue 15 Mar, 2005 17:28
Been thanked: 7 times
Contact:

Re: UpdateEx

Post by Alexey Samlyukov »

в связи с поддержкой ANSI плагов, сабжу все равно какой фар 1 или 3, он смотрит только на установленную версию и версию плагина на плагринге. вывод: пора забыть про старый фар и обновлять только уникодные версии плагинописателям, либо присваивать версии как в фаре: 1.75.ххх и 3.0.ххх.
User avatar
John Doe
Бюрократ
Posts: 13853
Joined: Wed 27 Apr, 2005 20:42
Has thanked: 73 times
Been thanked: 428 times
Contact:

Re: UpdateEx

Post by John Doe »

А почему бы просто не доработать апдейтер
User avatar
Alexey Samlyukov
Posts: 1765
Joined: Tue 15 Mar, 2005 17:28
Been thanked: 7 times
Contact:

Re: UpdateEx

Post by Alexey Samlyukov »

можно сделать проверку, если установлен анси, то и будем искать новое из анси.
сейчас можно обновить не зависимо от версии (анси->уникод).
делать?
User avatar
t-rex
Страшный и ужасный
Posts: 4910
Joined: Tue 15 Mar, 2005 16:17
Location: Tel-Aviv
Has thanked: 1 time
Been thanked: 8 times
Contact:

Re: UpdateEx

Post by t-rex »

нет, делать надо: если установлен юникод не апгрэйдим на анси.
Post Reply

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