Subzero: Improve the Doxygen input filter for TODO extraction.

Subzero: Improve the Doxygen input filter for TODO extraction.

The previous filter was too aggressive at changing "//" to "///".  The new filter does that transformation more selectively, starting at a commented TODO line and ending at the next non-commented line.

BUG= none

Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a=commit;h=e293b5f413b0fc9fed413bf5066ae3950704cc38

[Jim Stichnoth, 2015-12-09 00:17:09]

Description was changed from

==========
Subzero: Improve the Doxygen input filter for TODO extraction.

BUG= none
==========

to

==========
Subzero: Improve the Doxygen input filter for TODO extraction.

The previous filter was too aggressive at changing "//" to "///".  The new
filter does that transformation more selectively, starting at a commented TODO
line and ending at the next non-commented line.

BUG= none
==========

[Jim Stichnoth, 2015-12-09 00:17:09]

stichnot@chromium.org changed reviewers:
+ jpp@chromium.org, kschimpf@google.com, reed.kotler@imgtec.com,
sehr@chromium.org

[Jim Stichnoth, 2015-12-09 00:17:36]

Reed, please try this out and make sure I haven't missed anything this time.

[rkotlerimgtec, 2015-12-09 01:09:35]

On 2015/12/09 00:17:36, stichnot wrote:
> Reed, please try this out and make sure I haven't missed anything this time.

I did actually try it out last time, only I was not looking for collateral
damage.
I was just looking at the TODO lists and TODO inline part itself.

This patch I looked at a bunch of files and did not see any collateral damage
from the input filter.

LGTM.

[Jim Stichnoth, 2015-12-09 01:43:48]

On 2015/12/09 01:09:35, rkotlerimgtec wrote:
> On 2015/12/09 00:17:36, stichnot wrote:
> > Reed, please try this out and make sure I haven't missed anything this time.
> 
> I did actually try it out last time, only I was not looking for collateral
> damage.
> I was just looking at the TODO lists and TODO inline part itself.
> 
> This patch I looked at a bunch of files and did not see any collateral damage
> from the input filter.
> 
> LGTM.

Thanks, I didn't mean imply that it's anyone's fault but my own, just that
you're the expert among us on validating the results. :)

[Jim Stichnoth, 2015-12-09 01:44:40]

Description was changed from

==========
Subzero: Improve the Doxygen input filter for TODO extraction.

The previous filter was too aggressive at changing "//" to "///".  The new
filter does that transformation more selectively, starting at a commented TODO
line and ending at the next non-commented line.

BUG= none
==========

to

==========
Subzero: Improve the Doxygen input filter for TODO extraction.

The previous filter was too aggressive at changing "//" to "///".  The new
filter does that transformation more selectively, starting at a commented TODO
line and ending at the next non-commented line.

BUG= none

Committed:
https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a...
==========

[Jim Stichnoth, 2015-12-09 01:44:42]

Committed patchset #1 (id:1) manually as
e293b5f413b0fc9fed413bf5066ae3950704cc38 (presubmit successful).