Dave,

Thanks!  No rush.

There's another small problem we've encountered: when specifying the
--silence option the generated code misses commenting-out some lines.
Here's a snippet comprising incorrectly generated code:

def parse(inFilename):
    doc = minidom.parse(inFilename)
    rootNode = doc.documentElement
    rootObj = supermod.message.factory()
    rootObj.build(rootNode)
    # Enable Python to collect the space used by the DOM.
    doc = None
    #sys.stdout.write('<?xml version="1.0" ?>\n')
    #rootObj.export(sys.stdout, 0, name_="message",
        namespacedef_='')
    doc = None
    return rootObj


def parseString(inString):
    doc = minidom.parseString(inString)
    rootNode = doc.documentElement
    rootObj = supermod.message.factory()
    rootObj.build(rootNode)
    # Enable Python to collect the space used by the DOM.
    doc = None
    #sys.stdout.write('<?xml version="1.0" ?>\n')
    #rootObj.export(sys.stdout, 0, name_="message",
        namespacedef_='')
    return rootObj

We currently fix this manually by commenting out the namespacedef line.

Lou.

On Mon, Jun 22, 2009 at 2:42 PM, Dave Kuhlman <dkuhl...@pacbell.net> wrote:

> > When I use generateDS code to parse an xml instance all is well:
> >
> > <smc-config>
> >   <heu-config>
> >     <heu-def heu-num="1" heu-div="1" ip-address="10.0.0.1"
> host="localhost" port="8091" device-state="power-down" />
> >   </heu-config>
> > </smc-config>
> >
> > But when I use generateDS code to export an xml instance, all is NOT well
> because in some cases (for example heu-def) an underscore appears in place
> of the expected dash:
> >
> > <smc-config>
> >     <heu-config>
> >         <heu_def heu-num="1" heu-div="1" host="localhost"
> ip-address="10.0.0.1" port="8091" device-state="power-down">
> > </heu_def>
> >     </heu-config>
> > </smc-config>
> >
> > Am I doing something wrong?
> >
>
> Lou -
>
> No.  It's not you.  It's me.  I'm pretty sure that's a bug in
> generateDS.py.  It's worse.  I believe that I've made that
> mistake before.
>
> I have fixed this, now.  But, I'm in the middle of trying to fix
> another bug.  So, I won't be able to upload a new version for a
> couple of days.
>
> In the meantime, if you are in a hurry and need a fix now, you can
> probably fix it yourself.  Do either of the following:
>
> 1. Patch generateDS.py -- In functions generateExportFn_1(),
>    generateExportFn_2(), and generateExportFn_3(), look for places
>    where it is inserting "cleanName" (or maybe "mappedName"), and
>    change that so that it uses "name".  Look at the top of each of
>    these functions to see how these variables are set.  Or,
>
> 2. Patch the generated file -- Look in the exportChildren()
>    methods in the class heu_def, I believe.
>
> I'm hoping to have the real fix before the end of the week.  Sorry for
> the inconvenience.  I'll announce the new version as soon as it's
> available.
>
> And, thanks for catching this and alerting me to the problem.
>
> - Dave
>
>
>
------------------------------------------------------------------------------
Are you an open source citizen? Join us for the Open Source Bridge conference!
Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
Need another reason to go? 24-hour hacker lounge. Register today!
http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org
_______________________________________________
generateds-users mailing list
generateds-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/generateds-users

Reply via email to