This is the mail archive of the mailing list for the GCC project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: %pc relative addressing of string literals/const data

Mike Stump <> wrote on 2010/11/17 17:29:32:
> On Nov 17, 2010, at 1:22 AM, Joakim Tjernlund wrote:
> >> This patch is okay.
> >
> > Well, its been a week and I haven't seen any progress on this, let alone
> > my other patch which hasn't been commented upon at all.
> > What does "okay" really mean?
> Okay - Non-canonical spelling of Ok.

I never figured that :)

> Ok. -  Indication that a patch is ok to be checked in now;  no further work or discussion is necessary.  If you have write after privs, you can check it in (assuming you've met all other requirements, assignment, testing...).  If you don't have write privs, it means you can then solicit someone to
check it in for you.  The usual convention would be to address all other minor points people raised in review and repost and then check that in.  If there were major points raised during review, one more round of review and approval.  Essentially, if the new work would be obviously ok to the
maintainer, then you can skip the last review.  So for example, spelling corrections, white space fixups, wording in comments, avoidance of warnings would all be minor points.  A redo of an algorithm to be different, would require review.  If in doubt, ask Ok?  If you checked in it, say, committed
to trunk.  Absence either of these, leaves it unclear to the reviewers which state a discussion is in,
> though the usual convention would be to assume it was checked in.
> So, apparently, the right followup for you, would be the magic phrase, "Could someone check this in for me, thanks."  If you plan on doing a lot of work, and have submitted many patches that have made it in, the more appropriate phrase would be, "I don't yet have write-after privs, would someone
check this in for me, thanks."  This prods people to consider sponsoring you for a write-after account.
> That's the short version, I could give a much longer version, but, I'm lazy.  :-)

Right, me too.

There has been no objections, in fact the only comment I got is
"This patch is okay."
I do not have write privs of any kind nor do I want to have.
Could someone please check this patch in?


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]