This forum has been archived. Please start a
new discussion on GitHub
Another small manual fix
On page 1399, section 42.9.1 "Server Configuration" for IceStorm:
The property: IceBox.Service.IceStorm was updated for Ice 3.1 to use the new entry point createIceStorm, but the 30 needs to be updated to a 31, I believe.
Thanks muchly, we'll fix that for the next release.
I found another small one:
Section 29.10 on threads (page 712 in my 3.1.0 manual):
There is a small example about using the isAlive method. Namely, the line:
Isn't right because t is defined as a ThreadControl object and isAlive should be called directly on the thread.
Also, one clarification I think you may want to make (this one bit me for a couple of hours today). If you don't use a ThreadPtr to hold onto your thread, the internal implementation (which uses ThreadPtr) will delete the thread instance once it exits the run() method.
I was using a pimpl class:
class MyThread : public IceUtil::Thread
d = new MyThread; // d is a MyThread *
And it was failing on double destruction since MyThread deletes itself once run exits. Changing the code to use an IceUtil::Handle<MyThread> for d make it work.
I think it might be helpful to explicitly state this behavior in the docs.
Thanks for that, I'll fix this!