Stephan Bergmann wrote:
Some more input:

Hi Michael,

so i would be especially interested in input from people who would
like to use ODF metadata: does this API do what you need?

I don't know :), but have some other feedback (always "IMO")

- Pair< T, U > clearly belongs into css.beans
  (yes, that's nasty, since you need to add udkapi to your CWS then :)

Agreed, beans might be suitable.

- Not sure if the modifying methods of XRDFNamedGraph are necessary (or if they are only added to allow implementing XRDFRepository together with an "alien" XRDFNamedGraph implementation; in which case some private protocol between an XRDFRepository and the XRDFNamedGraph objects it creates might be better)---but I am absolutely not a domain expert with RDF.

- I vaguely remember I had some reason to consider UNOIDL constants, esp. "complex" ones like strings or (not even implemented) structs, as not useful, but unfortunately cannot remember the reason any longer...
as far as i remember they should be avoided especially string constants. In C++ they become static strings and i remember a performance issue with the static constructor.

Juergen


- Minor stylistic stuff:
-- XRDFURI vs. XRdfUri etc. (I personally prefer the latter, but the former is also widely used). -- If you are in module rdf, the "RDF" in exception, interface, and service names might be unnecessary.

-Stephan

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



--
Sun Microsystems GmbH        Juergen Schmidt
Nagelsweg 55                 Technical Lead Programmability
20097 Hamburg, Germany

Registered Office: Sun Microsystems GmbH, Sonnenallee 1, D-85551 Kirchheim-Heimstetten
Commercial register of the Local Court of Munich: HRB 161028
Managing Directors: Thomas Schroeder, Wolfgang Engels, Dr. Roland Boemer
Chairman of the Supervisory Board: Martin Haering

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to