Update SafeStack TODO in the safestack.cc
ClosedPublic
Actions

Authored by ksvladimir on Jun 22 2015, 5:08 AM.

Details

Reviewers

pcc
jfb

Commits

rGadbde27d2d1b: Update SafeStack TODO in the safestack.cc
rCRT240473: Update SafeStack TODO in the safestack.cc
rL240473: Update SafeStack TODO in the safestack.cc

Summary

This patch clarifies the TODO note at the top of safestack.cc and brings it more in sync with what we (the CPI team) actually plan to work on in the future.

Diff Detail

Event Timeline

ksvladimir updated this revision to Diff 28104.Jun 22 2015, 5:08 AM

ksvladimir retitled this revision from to Update SafeStack TODO in the safestack.cc.

ksvladimir updated this object.

ksvladimir edited the test plan for this revision. (Show Details)

ksvladimir added reviewers: pcc, jfb.

ksvladimir added a subscriber: Unknown Object (MLST).

Updated to include the cfe-commits list as a subscriber.

LGTM

This revision is now accepted and ready to land.Jun 23 2015, 2:41 PM

Closed by commit rL240473: Update SafeStack TODO in the safestack.cc (authored by pcc). · Explain WhyJun 23 2015, 3:31 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

lib/

safestack/

safestack.cc

56 lines

Diff 28104

View Options

lib/safestack/safestack.cc

Update SafeStack TODO in the safestack.ccClosedPublicActions