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

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.

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.  :-)

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