GitHub user zachhowe opened a pull request:
https://github.com/apache/thrift/pull/958
THRIFT-3750 Check isSet in copyWithZone
For each property being copied to the new object, ensure isSet is checked
before setting the new object's property. Otherwise isSet will always be YES on
new object even if it was not set to YES on the original object. This ensures
the new copied object is a true copy.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/zachhowe/thrift THRIFT-3750
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/thrift/pull/958.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #958
----
commit 39894afdd79fc54652e57afb4e42c6c871a9ea77
Author: Zach Howe <[email protected]>
Date: 2016-03-20T04:13:31Z
THRIFT-3750 Check isSet in copyWithZone
For each property being copied to the new object, ensure isSet is checked
before setting the new object's property. Otherwise isSet will always be YES on
new object even if it was not set to YES on the original object. This ensures
the new copied object is a true copy.
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
