Return-Path: Delivered-To: apmail-ws-axis-c-dev-archive@www.apache.org Received: (qmail 51996 invoked from network); 10 Jul 2006 12:21:03 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 10 Jul 2006 12:21:03 -0000 Received: (qmail 93749 invoked by uid 500); 10 Jul 2006 12:21:03 -0000 Delivered-To: apmail-ws-axis-c-dev-archive@ws.apache.org Received: (qmail 93723 invoked by uid 500); 10 Jul 2006 12:21:03 -0000 Mailing-List: contact axis-c-dev-help@ws.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: List-Post: List-Id: "Apache AXIS C Developers List" Reply-To: "Apache AXIS C Developers List" Delivered-To: mailing list axis-c-dev@ws.apache.org Received: (qmail 93712 invoked by uid 99); 10 Jul 2006 12:21:03 -0000 Received: from asf.osuosl.org (HELO asf.osuosl.org) (140.211.166.49) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 10 Jul 2006 05:21:03 -0700 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests= X-Spam-Check-By: apache.org Received-SPF: pass (asf.osuosl.org: local policy) Received: from [194.139.105.20] (HELO ns3.isogmbh.de) (194.139.105.20) by apache.org (qpsmtpd/0.29) with SMTP; Mon, 10 Jul 2006 05:21:02 -0700 Received: (qmail 15457 invoked from network); 10 Jul 2006 12:20:37 -0000 Received: from unknown (HELO ?185.2.10.89?) (192.168.81.11) by mail.dmz.isogmbh.de with SMTP; 10 Jul 2006 12:20:37 -0000 Message-ID: <44B245C4.7000207@isogmbh.de> Date: Mon, 10 Jul 2006 14:19:16 +0200 From: Franz Fehringer User-Agent: Thunderbird 1.5.0.4 (Windows/20060516) MIME-Version: 1.0 To: Apache AXIS C Developers List Subject: Re: handling of xml attributes flawed References: In-Reply-To: Content-Type: multipart/mixed; boundary="------------070702090907040603060903" X-ISOAntiVirus: scanned for viruses by ISO Mail-Scanner X-Spam-Checker-Version: SpamAssassin 3.0.2 (2004-11-16) on ns3.nbg.isogmbh.de X-Spam-Level: X-Virus-Checked: Checked by ClamAV on apache.org X-Old-Spam-Status: No, score=-104.4 required=5.0 tests=ALL_TRUSTED,AWL,BAYES_00, USER_IN_WHITELIST autolearn=ham version=3.0.2 X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N --------------070702090907040603060903 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Hello Adrian, Thanks for your explanations. I made local changes which essentially deems all attributes optional. For this i had to hack a significant amount of if statements to get a halfway reasonable code generation. I think this is way to crude to be of general interest and i have presently no more time to spend on this. But perhaps Nadir comes to the rescue? Greetings Franz Adrian Dick schrieb: > Franz, > > This is an area of code I haven't worked with for a little while now, so my > knowledge is a bit rusty, but I shall try and give some responses to your > questions below. > > Are ElementInfos for XML elements and AttributeInfos for XML > attributes or is this a misunderstanding on my side? > --> While I believe this was the original intent, taking a look > at the code it appears these concepts have become clouded as > the code has evolved -- although everytime I work in this area > I try to improve it a little. > In class Type elements maps names to ElementInfos, whereas > attributes maps names to Types. > Why not map names to AttributeInfos in the latter case? > --> I have a suspicion this could be because all (xml) elements > are stored in the typemap, so you use the name to lookup the > needed type, while attributes are not. > This seems to be the principal flaw to me; there is no direct > access to the attributes and their properties (isOptional for > example) in populateAttribList; only elements can be accessed > with their properties. > --> It does seem odd that we immediately work with the Type > while being able to work with AttributeInfo, that has already > been processed to give easy access to the desired properties, > would probably make more sense. > Now that my email has become enough confusing too, i think i will > give it a second try tomorrow. > As a last try for today are the following ideas sound? > AttributeInfos should be constructed from names and types like > ElementInfos (for attributes a simpler type concept than Type > would be sufficient). > If className (currently constructor argument for AttributeInfo) > is needed for source generation, it should be in the base class > ParamInfo. > --> This does seem odd, particularly as the classname used is > actually the one from the parent ParamInfo -- it really isn't > obvious to me how this is used. > In class Type, the attributes member should be a map from > String to AttributeInfo. > > Regards, > Adrian > _______________________________________ > Adrian Dick (adrian.dick@uk.ibm.com) > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: axis-c-dev-unsubscribe@ws.apache.org > For additional commands, e-mail: axis-c-dev-help@ws.apache.org > > --------------070702090907040603060903 Content-Type: text/x-vcard; charset=utf-8; name="feh.vcf" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="feh.vcf" begin:vcard fn:Dr. Franz Fehringer n:Fehringer;Franz org:ISO Software Systeme adr;quoted-printable:;;Eichendorffstrasse 29;N=C3=BCrnberg;;90491;Deutschland email;internet:mailto:feh@isogmbh.de tel;work:+49/(911) - 99594-0 tel;fax:+49/(911) - 99594-580 x-mozilla-html:TRUE url:http://www.isogmbh.de/ version:2.1 end:vcard --------------070702090907040603060903 Content-Type: text/plain; charset=us-ascii --------------------------------------------------------------------- To unsubscribe, e-mail: axis-c-dev-unsubscribe@ws.apache.org For additional commands, e-mail: axis-c-dev-help@ws.apache.org --------------070702090907040603060903--