Re: [jbosstools-dev] Weird code in Seam runtime management...

Max, how fast you forgot about org.jboss.tools.jst component :)

denis. Max Rydahl Andersen wrote: > Third question: > > Why is the seam facet id "jst.seam" ? > > "jst" is the name domain of Eclipse WTP's jst modules; what does our > seam facet have to do with that ? > > /max > >> Another question: >> >> Why do we need to have our own copy of application.xml when seam-gen >> has one ? >> >> We need to get seam-gen fixed if this is broken somehow. >> >> /max >> >>> Hi Denis, >>> >>> I stepped over some code I don't understand why is needed in Seam >>> runtime management. >>> >>> SeamRuntimeListConverter1 (weird class name, why an 1 at the end ?), >>> seem to do some really weird conversion between maps and strings on >>> the form >>> >>> "name|some Name|version|1123123|homeDir|somedir|default|true" >>> >>> ...why are we storing such weird strings in preferences ? >>> (Is that normal for eclipse preferences ? That looks very >>> unmaintainable) >>> >>> Why isn't the info about what runtime is default a single named >>> preference instead of maintained on each runtime ? >>> >>> And why are these classes in a package called >>> org.seam.core.project.facet ? These things are and should be >>> independent of WTP facets (and SeamRuntimeListConverter1 sound like >>> something that should be inside the internal package) >>> >>> /max >>> _______________________________________________ >>> jbosstools-dev mailing list >>> jbosstools-dev(a)lists.jboss.org >>> https://lists.jboss.org/mailman/listinfo/jbosstools-dev >> >> _______________________________________________ >> jbosstools-dev mailing list >> jbosstools-dev(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/jbosstools-dev >