Re: [Orbit-python-list] Uploaded patch

[Roland M. <ma...@ec...>]

Jason Tackaberry (2000-08-14 14:44:02 -0400) :

> I'm not sure, however, why you removed this from client.c:set_attribute():
> 
> if (ad->mode == CORBA_ATTR_READONLY) {
>    raise_system_exception("IDL:omg.org/CORBA/NO_PERMISSION:1.0", 0,
>                           CORBA_COMPLETED_NO, "attribute %s readonly",
>                           ad->name);
>    return -1;
> }

Um...  I'm not sure either :-)  Seriously, I probably cut too much by
mistake.  Sorry about that.

> Yes checks should be done at the server end as well (I'll merge that
> patch along with this one), but it certainly doesn't hurt to check
> at the client, since it will potentially eliminate needless
> communication with the server.

  Yep.  I'm eager to see it integrated, if you can figure out how to
send an exception, 'cause I couldn't.  Btw, the appropriate exception
would probably be NO_PERMISSION or NO_SUCH_METHOD (?), but I couldn't
find out which one either.

> >   Cool.  Thanks.  I'll probably build a Debian package for it, and
> > convert it to an RPM one (with alien).  I'll send them to you for
> > publishing on the Sourceforge download zone, if you like.
> 
> I would, yes.  Thanks. :)

  Just you release a tarball, I'll build and test the Debian package.
I can convert it to an RPM, but I have no Red Hat machine to test it,
so it will be untested.

  Ah, another thing.  Either I don't know how to free an object, or
there's another memory leak lurking somewhere.  I'm working on a test
suite I would be rather confident in, but in short and in my opinion,
the following code leaks memory (same ol' method, run it 100000 times
and count the bytes):

/---- CUT -----
| t2 = toto ()
| s2 = POA.truc.toto (t2)
| poa.activate_object (s2)
| ref = poa.servant_to_reference (s2)
| poa.deactivate_object (t2._servant)
\---- CUT -----

  I'm investigating into that, but if there's something known there
please tell me.

  And the last thing for this mail: I've been asked on
comp.lang.python if there were any plans to make ORBit-Python
OMG-compliant (about the mappings).  I quoted the webpage, but I'm not
really sure...  What is the status of that subproject?

Roland.
-- 
Roland Mas

[...] ou une dent pourrie [...] -- in Variations sur un thème imposé
  -- Signatures à collectionner, série n°2, partie 2/3.