-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hi Albrecht,

looks ok for me. Patch committed.


Thanks


Oliver


> Am 29.03.10 22:26 schrieb(en) Oliver Eichler:
>>> Not being a c++ expert, there is one thing which confused me: is the
>>> method run in COsmTilesHashCacheCleanup unused?  I changed the
>>> prototype, but noone complained...
>>
>> Ah, that won't work. See, the run() method is the thread that is
>> triggered by start(). Thus if you change API the default
>> implementation of run() is started instead of your run(QString
>> &forFolder). But as everything is within one class, and there is no
>> concurrent read/write access to the folder string, it should be ok to
>> share it as member variable, without further serialization.
> 
> Ooooops - as I said, I actually have no idea of c++ programming :-(  Sorry!
> 
> A new patch is attached, using a static class member and an accessor
> function instead.  Hope this solution is ok?  At least I always see the
> right path everywhere in gdb...
> 
> BTW, I noticed that Ubuntu purges the tmp folder upon each reboot. 
> Thus, it may be desirable to put the cache somewhere else.  The new
> patch now checks for an environment variable QLGT_CACHE, and if it is
> found, that location is used as cache folder.
> 
> Opinions?
> 
> Cheers, Albrecht.
> 
> 
> 
> ------------------------------------------------------------------------------
> Download Intel® Parallel Studio Eval
> Try the new software tools for yourself. Speed compiling, find bugs
> proactively, and fine-tune applications for parallel performance.
> See why Intel Parallel Studio got high marks during beta.
> http://p.sf.net/sfu/intel-sw-dev
> 
> 
> 
> _______________________________________________
> QLandkarte-users mailing list
> QLandkarte-users@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/qlandkarte-users


- -- 
- --------------------------------------------------------------------
DSP Solutions GmbH & Co. KG, Maximilianstraße. 16, 93059  Regensburg

Email: oliver.eich...@dspsolutions.de     http://www.dspsolutions.de
Phone: +49(0) 941 / 830 551               Embedded Signal Processing
Fax:   +49(0) 941 / 830 5579              DIN EN ISO 9001:2000

Gesellschaftssitz: Regensburg, Registergericht: Regensburg HRA  7862
Komplementaer: DSP Solutions Beteiligungs-GmbH, Regensburg HRB 10932
- --------------------------------------------------------------------
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.12 (GNU/Linux)
Comment: Using GnuPG with SUSE - http://enigmail.mozdev.org/

iEYEARECAAYFAkuy/t4ACgkQhhhJXTUlePp36ACff+Xbv84hU3CRlJGzvfh+3EAe
PJ4AoIbBjrJWld7kZl5+s0TV/o0AWTKi
=Ae4a
-----END PGP SIGNATURE-----

------------------------------------------------------------------------------
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
QLandkarte-users mailing list
QLandkarte-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/qlandkarte-users

Reply via email to