Fix bug in extractMaskFromPatch that masks from multiple ops are not merged correctly
commita650a9af94a99b9008fcd98b1459ffbff4805020
authorTJ Yin <ytj@meta.com>
Thu, 19 Jan 2023 09:02:30 +0000 (19 01:02 -0800)
committerFacebook GitHub Bot <facebook-github-bot@users.noreply.github.com>
Thu, 19 Jan 2023 09:02:30 +0000 (19 01:02 -0800)
treea704d5708d910c5c02b327c594e3666ea18dcab5
parentcb2a53055cd3b9a94ed0ed3f4fd132c6e20a10ac
Fix bug in extractMaskFromPatch that masks from multiple ops are not merged correctly

Summary: If patch has multiple ops, currently we just use the mask from last op overrides previous one. This diff changes to use unioned masks.

Reviewed By: rahku

Differential Revision: D42355484

fbshipit-source-id: e885ccb549bc6460f21c3e91e468ef0634d54ba9
third-party/thrift/src/thrift/lib/cpp2/protocol/Patch.cpp
third-party/thrift/src/thrift/lib/cpp2/protocol/PatchTest.cpp