[Lazarus] Rescan FPC sources crashes Lazarus

[Mattias Gaertner]

On Fri, 25 Mar 2011 23:19:48 +0200
Graeme Geldenhuys <graemeg.lists at gmail.com> wrote:

> On 25 March 2011 23:08, Mattias Gaertner <nc-gaertnma at net****> wrote:
> >
> > The scan uses multi threading, which was broken by the installed
> > package.
> 
> I was reading the thread and replying before I read the whole thread.
> Indeed that installed package did do something it was not supposed to
> do.
> 
> In all fairness, how are all developers supposed to know that? 

If you don't know the purpose of a variable, why change it?
The documentation and comment do not say that you should set it.

> Isn't
> there a better way that FPC could hide such functionality (not
> allowing a developer to set a RTL controlled variable they are not
> supposed do). That should really be filed as a bug against FPC. They
> [fpc] should rather use some singleton class with a read-only
> property, or simply a function (function IsRTLMultiThreaded: boolean;)
> or something, not just a global read-write variable - that's just
> sloppy!

AFAIK when writing your own thread manager you must set it.


Mattias