On Mon, Feb 12, 2007 at 09:27:11PM -0600, David Lambert wrote:
> Dave,
>    Thanks for you fix - all seems to be as expected now. I have one 
> more question, however. The methods set.. and get... appear to lose the 
> case of the original name. For instance I have an element named  
> 'DeepReceiverUp' but I note that the methods are named 
> setDeepreceiverup(), and getDeepreceiverup(). Could you please let me 
> know if this is intentional, and if so why is the case lost? It is not a 
> huge problem, but I am just curious :-)

David -

It's a blemish.  It's a decision that I made a long time ago and
wish I had made differently.  If I had it to do over, I'd generate:

    get_xyz()
    set_xyz()

which is more common in the Python world, I believe, rather than

    getXyz()
    setXyz()

Hopefully you've prodded me to finally change this.  The next 24
hours or so will tell.

Suppose that I modify generateDS.py so that:

1. generateDS.py generates getters and setters of the form
   get_varname() and set_varname() instead of getVarname() and
   setVarname() and

2. I add a switch so that the old behavior can also be requested.

Does that seem like an improvement to you?

[a bit later ...]

It looks do-able, and was a good deal easier than I expected.  I
believe I've got most of the change made.  Give me until tomorrow
to do more testing, document it, package it, etc.

But, I would appreciate your opinion before committing it.  Thanks.

Dave K.


-- 
Dave Kuhlman
http://www.rexx.com/~dkuhlman

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier.
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
generateds-users mailing list
generateds-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/generateds-users

Reply via email to