[wxPython-dev] img2py script

[Anthony Tuininga]

With apologies for the delay...I have finally managed to integrate the
changes you suggested and upload a new unified patch file for all of
the changes made to the wx.lib and wx.tools subdirectories. If you
have additional changes you would like to make, please let me know.
Otherwise, once I see the changes checked in, I'll make similar
changes for the demo and other tools. Thanks to your patience. I hope
you all had a good Christmas.

Anthony

On Dec 13, 2007 3:27 PM, Robin Dunn <robin at alldunn.com> wrote:
>
> Robin Dunn wrote:
> > Anthony Tuininga wrote:
> >> On Dec 10, 2007 11:53 AM, Robin Dunn <robin at alldunn.com> wrote:
> >>> Anthony Tuininga wrote:
> >>>> Hi, I believe you've been a little busy lately with the latest release
> >>>> of wxPython but I was wondering if you'd had a chance to look at this
> >>>> at all?
> >>> A little bit so far, hopefully I'll get to it in more depth this week.
> >>
> >> Sure. I'm in no particular rush so long as it does get done
> >> eventually. :-)
> >>
> >>>> I'm still waiting before modifying the demo. I've noticed a
> >>>> few other places that could use the same treatment (like Editra and
> >>>> XRCed) which I can also change if that is desirable. Let me know.
> >>> Yep, the less work for me the better.  ;-)
> >>
> >> Then I'll do that after I've heard from you about the base code.
> >
> > Just noticed a problem...  If the image name is not a valid Python
> > identifier then img2py can generate bad code.  For example, in
> > throbImages.py we end up with this:
> >
> >
> > 001 = PyEmbeddedImage(
> >     ...
> >     )
> > index.append('001')
> > catalog['001'] = 001
> > get001Data = 001.GetData
> > get001Image = 001.GetImage
> > get001Bitmap = 001.GetBitmap
> >
> > I think we should still use the given name for the index and catalog,
> > but there needs to be some sort of name fixup for the instance variable
> > name if it would cause a problem like this.  Maybe prepending a "_" if
> > the name starts with a digit, and also something needs to be done if
> > there are non-ascii characters or spaces in the name.
> >
> >
>
>
> Also, I would prefer to have all the diffs in one patch file.  That way
> I only have to run patch once.
>
>
>
> --
> Robin Dunn
> Software Craftsman
> http://wxPython.org  Java give you jitters?  Relax with wxPython!
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: wxPython-dev-unsubscribe at lists.wxwidgets.org
> For additional commands, e-mail: wxPython-dev-help at lists.wxwidgets.org
>
>