opensubscriber
   Find in this group all groups
 
Unknown more information…

c : classpath-patches@gnu.org 16 March 2011 • 6:15AM -0400

Re: [cp-patches] [PATCH] Fix Use ArrayList.add() in StyleSheet.resolveStyle
by Dr Andrew John Hughes

REPLY TO AUTHOR
 
REPLY TO GROUP




On 12:55 Tue 15 Mar     , Pekka Enberg wrote:
> Hi Andrew,
>
> On Tue, Mar 15, 2011 at 12:47 PM, Dr Andrew John Hughes
> <ahughes@redh...> wrote:
> > Please just make it clear what you're asking for.  If it's a fix
> > you're sure about, then just post to the patches
> > list with the FYI: subject prefix and commit straight away.  If not,
> > prefix the subject with RFC: and wait
> > for feedback.
> >
> > With this one, it was unclear whether you wanted feedback or not and
> > the commit wasn't immediate, but delayed by two days.
>
> Oh, sorry about that - I'm still in my 'other project' mindset. I was
> thought the fix was obvious enough but delayed the commit by few days
> so people could take a look at the patch to see if you wanted me to
> fix it in some other way.
>

Yeah it's just a bit confusing for anyone (probably just me...)
watching that process as you didn't reply to say why you were
committing...

> /me goes to re-read the GNU Classpath Hacker's Guide...
>
>                         Pekka

--
Andrew :)

Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

Support Free Java!
Contribute to GNU Classpath and IcedTea
http://www.gnu.org/software/classpath
http://icedtea.classpath.org
PGP Key: F5862A37 (https://keys.indymedia.org/)
Fingerprint = EA30 D855 D50F 90CD F54D  0698 0713 C3ED F586 2A37


Bookmark with:

Delicious   Digg   reddit   Facebook   StumbleUpon

Related Messages

opensubscriber is not affiliated with the authors of this message nor responsible for its content.