make SyncRequest struct public
ClosedPublic

Authored by aseigo on Jul 28 2017, 11:58 AM.

Details

Summary

I'm assuming this is a compiler age issue, but when trying to access the SyncRequest struct in a lambda it seems to use the lambda's magic C++ class definition as scope and loses the ability to access the struct due to being protected.

This works around that blemish, while also removing a bunch of extra protected declarations.

Diff Detail

Repository
R9 Sink
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
aseigo created this revision.Jul 28 2017, 11:58 AM
cmollekopf accepted this revision.Jul 29 2017, 12:46 AM
This revision is now accepted and ready to land.Jul 29 2017, 12:46 AM
This revision was automatically updated to reflect the committed changes.