Archived
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.
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.
0
Comments
-
Thanks muchly, we'll fix that for the next release.
Cheers,
Michi.0 -
Hi Michi,
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:
t->isAlive()
Isn't right because t is defined as a ThreadControl object and isAlive should be called directly on the thread.0 -
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
{
blah;
};
MyClass::MyClass()
{
d = new MyThread; // d is a MyThread *
}
MyClass::~MyClass()
{
delete d;
}
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!0 -
Thanks for that, I'll fix this!
Cheers,
Michi.0