On Tuesday 24 April 2012 21:23:08 Marcos Douglas wrote:
> >
> > Maybe it is time now to fork dp.pas.
> > Please use FPC fixes_2_6 SVN revision 20423, it is the last working.
> >
>
> You mean DB.pas... yes, maybe.
>
No, db.pas:
http://svn.freepascal.org/cgi-bin/viewvc.cgi/tags/release_2_6_0/packages/fcl-db/src/base/db.pas?revision=19802&view=markup
It could break integration of third party DB components (ZEOS) into 
MSEide+MSEgui.

> Thanks, but I'm already in 21016 rev. I will wait what do you want to
> do about it (fork or adapt the sources to use the (re)new approach).
>
If it stays as it is now in FPC fixes_2_6 you must change every assignment of 
TDataSet.Bookmark in your code.
Old:
"
var
 bm: tbookmarkstr;
[...]
 bm:= <tdataset>.bookmark;
 //do something with <tdataset>
 <tdataset>.bookmark:= bm; //restore DB cursor
"

New:
"
var
 bm: tbookmark;
[...]
 bm:= <tdataset>.bookmark;
 try
  //do something with <tdataset>
  <tdataset>.bookmark:= bm; //restore DB cursor
 finally
  <tdataset>.freebookmark(bm); //avoid memory leak
 end;
"
Sometimes I don't understand FPC people...

Martin

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
mseide-msegui-talk mailing list
mseide-msegui-talk@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mseide-msegui-talk

Reply via email to