Достаточно часто (но не всегда) в одном и том же месте появляется ошибка.
Access violation at address .... in module 'MyExe.exe'. Read of address 0000000F.

WThread.TWThread.PostToThreadMessage (Line 393, "WThread.pas") - по EurekaLog

версия от 28.07.2014
result := (not Suspended)and(PostThreadMessage(ThreadID, Msg, wParam, lParam));

Не знаю как лечить. Может быть поможете?
Раньше использовали предыдущие версии и все нормально работало.

wadman,

сейчас отправлю в личку.

А так, вкратце там выводится форма с часиками (что типа что-то выполняется и ждите). форма нарисована на winapi. с помощью вашего класса все данные на этой форме обновляются.

wadman,

что-то лички тут не нашел, ладно, тогда так отправлю

wadman,

Ну там все намного проще. раньше все это было написано как класс, в котором запускается отдельной процесс с какой-то логикой. То есть был основной поток с программой, из основного потока создавался поток с нужной процедурой и еще один поток для отрисовки окна.
все работало нормально, без каких либо ошибок и все было хорошо. Но пришлось все переписывать, так как если в этом отдельном потоке создавался fastreport с поддержкой мультизадачности (EnableThreadSafe = False), то по полной начинало все глючить.
И из-за этого было быстренько все переписано и теперь поток не создается с нужной процедурой, а выполняется в основном потоке. А для отрисовки окна используется Ваш класс. Да, переписано все через ж.., но к сожалению что есть, то и есть. Вначале писал я (с кучей поток), переписывал не я, а человек, который уже был уволен за кучу своих ошибок.
N-ая часть этого кода из приложенного файла не используется уже совсем, валяется как мусор или пережиток былого. А почистить времени нет.

> На сколько я понял - это тяжелое наследие какого-то участка кода, который когда-то работал в одном потоке и теперь стоит задача разнести на разные?
Наоборот, был когда то перевод с кучи потоков на один поток (ну +1 на форму с "часиками")

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

> Например: Под параметры память выделяется, а под текст - нет. См. функции NewString, FreeString.
раньше так все и было, сейчас это не нужно

Просто опять же, раньше на какой-то другой версии Вашего класса ошибка не возникала, сейчас возникает. Вот стало интересно почему, отсюда и спросил

P.S.: А так, по существу, как то никогда не было проблем как с потоками, так и с дин памятью, так и с тем, как с ней работать

wadman,

код экспорта был таким же с точностью до байта, как и в тех версиях Вашего класса. Кстати еще раз, это не экспорт, а выполнение любых процедур, когда заранее точно не известно выполнится ли эта процедура быстро или нет.
На тему Procedure TTimerThread.UpdateForm; - он не используется совсем у меня. Как уже сказал, остались части, надо вычистить.

На тему слипа. там идет основной поток программы, а раз так, то могу делать в основном потоке что хочу. Разве нет?

На тему ThreadParams^.MessageText := MessageText;
ThreadParams - это рекорд, под который выделена память с помощью new. По моему я уже выделил память, в том числе и под строку. А работаю исключительно с указателем на этот рекорд и эту строку. Разве нужно еще раз выделять память под строку уже в выделенной памяти под весь рекорд?

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

> Скорее всего поток к тому времени не создан, либо убит (то есть равен nil) и обращение к ThreadID вызывает ошибку.
вполне возможно, надо попробовать поставить проверку

wadman,

> String по умолчанию уже давно лишь ссылка на участок памяти с символами
конечно я это знаю

> Вот тебе и понимание работы с памятью и потоками
оке, тогда объясни что не так? и почему огромное количество времени (с 2006 по 2013 год) предыдущий вариант этого класса (моего класса формы с часами) работал БЕЗ ЕДИНОЙ ПРОБЛЕМЫ?
проблема с EnableThreadSafe = false не в счет, тут уже прикол самого фастрепорта, как Вы сами написали.

wadman,

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