User forums > Nightly builds

The 25 August 2007 build will NOT be out.

(1/2) > >>

killerbot:
first we need to sort out this : http://forums.codeblocks.org/index.php?topic=6765.msg51964#msg51964

thomas:
Martin reverted it though? So... nothing to solve?

stahta01:
SVN 4410 crashes on my WinXP SP2, wxWidgets 2.8 Branch

Tim S

PS: Trying backgroundthread.h SVN 4181 now to see if it works OK. It seemed to fix the Crash on startup issue. Note, I only changed the one file from SVN 4410.

The patch to revert backgroundthread.h to SVN 4181 is below.

--- Code: ---Index: src/include/backgroundthread.h
===================================================================
--- src/include/backgroundthread.h (revision 4410)
+++ src/include/backgroundthread.h (working copy)
@@ -27,18 +27,13 @@
 * BackgroundThreadPool is a low overhead thread pool implementation around BackgroundThread.
 */
 
+
 class AbstractJob
 {
-    friend class BackgroundThread;
-
-    bool deleted;
-    bool done;
-
 public:
-    AbstractJob() : deleted(false), done(false) {};
+    AbstractJob(){};
     virtual ~AbstractJob(){};
     virtual void operator()() = 0;
-    void Delete() { deleted = true; if(done) delete this;};
 };
 
 
@@ -155,20 +150,10 @@
                 break;
 
             job = queue->Pop();
-
-            if(job->deleted)
-            {
-                delete job;
-                continue;
-            }
-
             (*job)();
-            job->done = true;
 
-            if(job->deleted || ownsJobs)
-            {
+            if(ownsJobs)
                 delete job;
-            }
         }
         return 0;
     };

--- End code ---

killerbot:
no It was not reverted. And as stated in an other thread, the precompiled header include should be removed (this will not cause a crash, I think, but can lead to 'not building').

MortenMacFly:

--- Quote from: thomas on August 26, 2007, 12:18:04 am ---Martin reverted it though? So... nothing to solve?

--- End quote ---
No, I didn't revert. I don't revert code I don't understand - and I was under the assumption you had a good reason for your revert...?!

Navigation

[0] Message Index

[#] Next page

Go to full version