FileCopyEx: File Copy Extended — Расширенное копирование

Здесь можно обсудить ваш любимый плагин.
User avatar
chupakabra
Posts: 1023
Joined: Wed 27 Jul, 2005 19:36
Contact:

Re: FileCopyEx: Extended File Copy — Расширенное копирование

Post by chupakabra »

Latest post of the previous page:

теперь память надо выделять самому, и хз когда ее убивать. поэтому я с этим не заморачивался
:shock:
djdron
Posts: 32
Joined: Mon 13 Dec, 2010 22:33

Re: FileCopyEx: Extended File Copy — Расширенное копирование

Post by djdron »

chupakabra, да, 50 строк - это дохера утечек, согласен :)
User avatar
DrKnS
Posts: 6114
Joined: Thu 04 Aug, 2005 06:44
Location: Kyiv
Has thanked: 12 times
Been thanked: 426 times

Re: FileCopyEx: Extended File Copy — Расширенное копирование

Post by DrKnS »

:facepalm:
djdron
Posts: 32
Joined: Mon 13 Dec, 2010 22:33

Re: FileCopyEx: Extended File Copy — Расширенное копирование

Post by djdron »

не кручиньтесь, парни ;-) вот версия без этих утечек!
djdron
Posts: 32
Joined: Mon 13 Dec, 2010 22:33

Re: FileCopyEx: Extended File Copy — Расширенное копирование

Post by djdron »

sonar, затести, плз
User avatar
ctapmex
Posts: 1240
Joined: Fri 11 Apr, 2008 07:53
Has thanked: 1 time
Been thanked: 50 times

Re: FileCopyEx: Extended File Copy — Расширенное копирование

Post by ctapmex »

стало лучше. )
но там "утечки " изначальные, т.е. не добавленные тобой. и связаны большей частью с не удалением памяти при выходе.
пока со временем напряг, ближе к новогодним праздникам погляжу внимательнее, приложу патчи
User avatar
ctapmex
Posts: 1240
Joined: Fri 11 Apr, 2008 07:53
Has thanked: 1 time
Been thanked: 50 times

Re: FileCopyEx: Extended File Copy — Расширенное копирование

Post by ctapmex »

вот патч. не претендует на идеальность, но указывает основные проблемные места.
Attachments
37.zip
(1.2 KiB) Downloaded 107 times
djdron
Posts: 32
Joined: Mon 13 Dec, 2010 22:33

Re: FileCopyEx: Extended File Copy — Расширенное копирование

Post by djdron »

sonar, спасибо.
смотрю тут работу с объектами... ппц... все раскидано по туче файлов.
ну ничего, скоро упорядочим.
кстати, даже с твоим патчем есть место где объекты удаляются через delete void* указателя - следовательно не вызовется деструктор и не почистятся все мемберы.
это в StringList::DeletePtr();
смотрю как его вообще выкинуть - слишком наворочено все, темплейты от 3х параметров :shock: :crazy:
Kildor
Posts: 1515
Joined: Wed 16 Mar, 2005 06:26
Location: Новосибирск
Been thanked: 1 time
Contact:

Re: FileCopyEx: Extended File Copy — Расширенное копирование

Post by Kildor »

WinXP. Far 1766.
Последняя версия плагина отказывается копировать в связи, утверждая что «путь назначения не верен»
Ты мне не выкай!
«ФарФорум не для пользователей!»
Upd:
P.S. Я просто поддерживаю беседу.» © Народ.

xmpp:kildor@jabber.ru
djdron
Posts: 32
Joined: Mon 13 Dec, 2010 22:33

Re: FileCopyEx: Extended File Copy — Расширенное копирование

Post by djdron »

Kildor, версия которая rev42?

Можно полную последовательность действий, а то я попробовал у себя - копирует.
User avatar
ctapmex
Posts: 1240
Joined: Fri 11 Apr, 2008 07:53
Has thanked: 1 time
Been thanked: 50 times

Re: FileCopyEx: Extended File Copy — Расширенное копирование

Post by ctapmex »

Лучше стало. но не до конца. два проблемных места
1.dlgobject.cpp строка 97 - 22 раза там побывали (при моем тесте) и не почистили
2. dlgclass.cpp строка 68 - 3 раза
djdron
Posts: 32
Joined: Mon 13 Dec, 2010 22:33

Re: FileCopyEx: Extended File Copy — Расширенное копирование

Post by djdron »

sonar, спс! добавил проверку на crt memory leaks и убил их всех :)
djdron
Posts: 32
Joined: Mon 13 Dec, 2010 22:33

Re: FileCopyEx: Extended File Copy — Расширенное копирование

Post by djdron »

очередной nightly build :-)

+ убраны утечки памяти
+ отключены exceptions & rtti, т.к. не используются
+ уменьшены размеры .dll (следствие предыдущего пункта)
+ настроены флаги компилятора на полную оптимизацию
User avatar
ctapmex
Posts: 1240
Joined: Fri 11 Apr, 2008 07:53
Has thanked: 1 time
Been thanked: 50 times

Re: FileCopyEx: Extended File Copy — Расширенное копирование

Post by ctapmex »

сейчас чисто. спасибо за поддержку плагина .
Anax
Posts: 5
Joined: Fri 04 Dec, 2009 11:39

Re: FileCopyEx: Extended File Copy — Расширенное копирование

Post by Anax »

У меня настройки почему-то стали сохраняться в какое-то странное место:
HKEY_CURRENT_USER\Software\Far2\Plugins\UnknownPlugin

И при параллельном копировании больших файлов дескрипторы утекают.
Post Reply

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