> 2. Why did you add 'Exception' to some method signatures?  for example:
>      throws WSDLException, JAXBException, Exception {
>
> Now the caller is forced to catch Exception? Throwing Exception hides granularity, and can make it harder to debug, or to catch appropriate exceptions.

This is for wsdl2uddi and for supporting a Web accessible wsdl via Apache http client which throws a wide variety of exceptions

OK still not sure what you mean with that; you can always catch Exception to catch them all.

> 4. You added 2 convenience constructors in the uddi-ws module on UDDI v3 entities. One should be able to use their own UDDIv3 generated classes. So in theory this would break that. In practice at the moment we've added our own juddiAPI classes so we've already broken it, but I think we should try to remove those classes at some point if possible. Also adding javadoc to these files is nice, but if we regenerate then that will all be last again, unless we manually add it back in.
>

You said adding java doc was ok. With it, there's no value added to using the uddiws library. As you said, its already broken. Jboss esb requires the juddi version of uddiws. I've replaced it with my own generated from the wsdl and it broke. Do ugh its already broken we might as well make it useful

Given some time I'd rather fix it.

> 7. I don't think you meant to check in the Doxyfile in the root?

I did

OK we don't check in any IDE specific files. Everyone has their own little setup. You probably want to add it to .svnignore

Reply via email to