[prev in list] [next in list] [prev in thread] [next in thread] 

List:       koffice-devel
Subject:    Re: [patch] exp0rt method rename
From:       Sebastian Sauer <mail () dipe ! org>
Date:       2008-03-09 1:28:29
Message-ID: 200803090228.29344.mail () dipe ! org
[Download RAW message or body]

On Sunday 09 March 2008, Kevin Krammer wrote:
> On Sunday 09 March 2008, Sebastian Sauer wrote:
> > On Saturday 08 March 2008, Kevin Krammer wrote:
> > > the attached patch gets rid of the IMHO rather ugly "leet speak"
> > > exp0rt() name by changing the method name to exportDocument()
> > > Also changes import to importDocument() for consistency.
> > >
> > > Might not have caught all occurencies of import() since I don't have
> > > all the optional libraries for certain plugins installed.
> >
> > Guess it's really not needed to pre-review each patch, specialy since
> > your patches normaly rock anyway ;)
>
> Thanks :)
>
> > So, probably just commit and let others
> > post-review them if they have such a desire?
>
> OK, comitted.
>
> Since this is the first time I am changing code in KOffice, I take it is a
> matter of politeness to basically "ask" for commit permission.

y, true. It may help to avoid strange situations. Guess I should adopt that 
behaviour. Thanks for the hint :)
_______________________________________________
koffice-devel mailing list
koffice-devel@kde.org
https://mail.kde.org/mailman/listinfo/koffice-devel
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic