Ah yes, I almost forgot about that one.
James Mitchell Software Engineer/Struts Evangelist http://www.open-tools.org "If you were plowing a field, which would you rather use? Two strong oxen or 1024 chickens?" - Seymour Cray (1925-1996), father of supercomputing > -----Original Message----- > From: Craig R. McClanahan [mailto:craigmcc@;apache.org] > Sent: Tuesday, November 12, 2002 6:33 PM > To: Struts Developers List > Subject: RE: Replacing Action.* with Globals.******** README ********* > > > +1, with only one additional reminder -- convert leading tabs to spaces as > well. > > Craig > > > On Tue, 12 Nov 2002, Martin Cooper wrote: > > > Date: Tue, 12 Nov 2002 13:27:24 -0800 (PST) > > From: Martin Cooper <[EMAIL PROTECTED]> > > Reply-To: Struts Developers List <[EMAIL PROTECTED]> > > To: Struts Developers List <[EMAIL PROTECTED]> > > Subject: RE: Replacing Action.* with Globals.******** README ********* > > > > > > > > On Tue, 12 Nov 2002, James Mitchell wrote: > > > > > Since we are all bashing David (LOL...just kidding), may I > make one more > > > suggestion. > > > > > > Can we all agree to separate code changes with format changes? > > > > +1 > > > > > > > > I usually check every commit to see if it impacts something I might be > > > working on (bug fix or personal dev) and as I was browsing my > mail this > > > morning, I found that reading through a couple of "recent > commits " <wink/> > > > was giving me a migraine. Finding the actual code changes > was impossible. > > > > > > In fact, we probably should not be using our IDE's 'reformat' features > > > unless the current page formatting is complete crap. If we > each reformat > > > every time we open and change a file. We will see battles > emerging between > > > JBuilder, NetBeans, Eclipse, and others. > > > > +1 > > > > > > > > If that sounds feasible, I would also recommend to those of > us just using > > > text editors to turn off the 'trim trailing spaces' when saving. This > > > causes just as much of a headache. > > > > +1 > > > > JBuilder is also a big culprit here. > > > > > > > > Your thoughts? > > > > Basically, I couldn't agree more. ;-) > > > > -- > > Martin Cooper > > > > > > > > > > > > > > > > James Mitchell > > > Software Engineer/Struts Evangelist > > > http://www.open-tools.org > > > > > > "If you were plowing a field, which would you rather use? Two > strong oxen or > > > 1024 chickens?" > > > - Seymour Cray (1925-1996), father of supercomputing > > > > > > > > > > -----Original Message----- > > > > From: David Graham [mailto:dgraham1980@;hotmail.com] > > > > Sent: Tuesday, November 12, 2002 12:10 PM > > > > To: [EMAIL PROTECTED] > > > > Subject: RE: Replacing Action.* with Globals.* > > > > > > > > > > > > You are right. I didn't wait long enough because I was making > > > > other changes > > > > and didn't want to get confused about which ones to commit. > > > > > > > > David > > > > > > > > > > > > > > > > > > > > > > > > > > > > >From: "Martin Cooper" <[EMAIL PROTECTED]> > > > > >Reply-To: "Struts Developers List" <[EMAIL PROTECTED]> > > > > >To: "'Struts Developers List'" <[EMAIL PROTECTED]> > > > > >Subject: RE: Replacing Action.* with Globals.* > > > > >Date: Tue, 12 Nov 2002 08:49:49 -0800 > > > > > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > From: David Graham [mailto:dgraham1980@;hotmail.com] > > > > > > Sent: Tuesday, November 12, 2002 8:30 AM > > > > > > To: [EMAIL PROTECTED] > > > > > > Subject: Re: Replacing Action.* with Globals.* > > > > > > > > > > > > > > > > > > I committed the updates last night because I didn't hear > > > > > > anyone complain. > > > > > > > > > >I don't have a problem with the changes. > > > > > > > > > >I would like to point out, however, that you only waited 3 > hours for > > > > >feedback. Not all of us are in the same country, let alone > the same time > > > > >zone. Cedric, for example, had no hope of responding in > time, unless he > > > > >happened to be reading Struts mail at around 3am his time > (my estimate). > > > > >Even those of us in an appropriate time zone are not all constantly > > > > >checking > > > > >mail to struts-dev. > > > > > > > > > >Given your statement that you "would hate to do this and > have to back out > > > > >the changes" (which I understand :), I would suggest that you > > > > might want to > > > > >allow more time for people to respond before going ahead with > > > > changes like > > > > >this. Otherwise, you do leave yourself open to having to > back out the > > > > >changes. > > > > > > > > > >-- > > > > >Martin Cooper > > > > > > > > > > > > > > > > > > > > > > David > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >From: Eddie Bush <[EMAIL PROTECTED]> > > > > > > >Reply-To: "Struts Developers List" > <[EMAIL PROTECTED]> > > > > > > >To: Struts Developers List <[EMAIL PROTECTED]> > > > > > > >Subject: Re: Replacing Action.* with Globals.* > > > > > > >Date: Tue, 12 Nov 2002 09:41:04 -0600 > > > > > > > > > > > > > >Well, we're headed toward using Globals instead of Action > > > > > > for this purpose, > > > > > > >so I'd think it would be highly desirable for the internal > > > > > > code to use > > > > > > >those constants located in Globals. > > > > > > > > > > > > > >... The Action constants can't go away yet (at least I > > > > > > believe that's the > > > > > > >decision that was arrived at), but I can't see where > > > > > > changing internal code > > > > > > >to use Global references over Action references would > be a bad thing. > > > > > > > > > > > > > >David Graham wrote: > > > > > > > > > > > > > >>Is it ok to replace the Action constant references with > > > > > > Globals constant > > > > > > >>references in the Struts code? I would hate to do this and > > > > > > have to back > > > > > > >>out the changes. > > > > > > >> > > > > > > >>Thanks, > > > > > > >>Dave > > > > > > > > > > > > > > > > > > > > >-- > > > > > > >Eddie Bush > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >-- > > > > > > >To unsubscribe, e-mail: > > > > > > ><mailto:struts-dev-unsubscribe@;jakarta.apache.org> > > > > > > >For additional commands, e-mail: > > > > > > ><mailto:struts-dev-help@;jakarta.apache.org> > > > > > > > > > > > > > > > > > > > _________________________________________________________________ > > > > > > MSN 8 helps eliminate e-mail viruses. Get 2 months FREE*. > > > > > > http://join.msn.com/?page=features/virus > > > > > > > > > > > > > > > > > > -- > > > > > > To unsubscribe, e-mail: > > > > > > <mailto:struts-dev-unsubscribe@;jakarta.apache.org> > > > > > > For additional commands, e-mail: > > > > > > <mailto:struts-dev-help@;jakarta.apache.org> > > > > > > > > > > > > > > > > > > > > > > > > > > >-- > > > > >To unsubscribe, e-mail: > > > > ><mailto:struts-dev-unsubscribe@;jakarta.apache.org> > > > > >For additional commands, e-mail: > > > > ><mailto:struts-dev-help@;jakarta.apache.org> > > > > > > > > > > > > _________________________________________________________________ > > > > Add photos to your messages with MSN 8. Get 2 months FREE*. > > > > http://join.msn.com/?page=features/featuredemail > > > > > > > > > > > > -- > > > > To unsubscribe, e-mail: > > > <mailto:struts-dev-unsubscribe@;jakarta.apache.org> > > > For additional commands, e-mail: <mailto:struts-dev-help@;jakarta.apache.org> > > > > > > > > -- > > To unsubscribe, e-mail: <mailto:struts-dev-unsubscribe@;jakarta.apache.org> > > For additional commands, e-mail: <mailto:struts-dev-help@;jakarta.apache.org> > > > > > > > -- > To unsubscribe, e-mail: <mailto:struts-dev-unsubscribe@;jakarta.apache.org> > For additional commands, e-mail: <mailto:struts-dev-help@;jakarta.apache.org> > > -- To unsubscribe, e-mail: <mailto:struts-dev-unsubscribe@;jakarta.apache.org> For additional commands, e-mail: <mailto:struts-dev-help@;jakarta.apache.org> -- To unsubscribe, e-mail: <mailto:struts-dev-unsubscribe@;jakarta.apache.org> For additional commands, e-mail: <mailto:struts-dev-help@;jakarta.apache.org>
