[
https://issues.apache.org/jira/browse/THRIFT-5463?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jens Geyer resolved THRIFT-5463.
--------------------------------
Fix Version/s: 0.25.0
Assignee: Sylwester Lachiewicz
Resolution: Fixed
> Incorrect and inconsistency in compiler generated go code regarding pointer
> types
> ---------------------------------------------------------------------------------
>
> Key: THRIFT-5463
> URL: https://issues.apache.org/jira/browse/THRIFT-5463
> Project: Thrift
> Issue Type: Bug
> Components: Go - Compiler
> Affects Versions: 0.15.0, 0.14.2
> Reporter: Yuxuan Wang
> Assignee: Sylwester Lachiewicz
> Priority: Major
> Fix For: 0.25.0
>
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
> This is the continuation/followup from THRIFT-5389 and THRIFT-5461.
> # For container fields of a struct (map/list/set), we make the container
> type a point when it has default value set, but we didn't handle that
> correctly when generating the constant value (part of THRIFT-3289 but missed
> from the fix)
> # The decision on whether to make string/binary fields of a struct's type a
> pointer is inconsistent from container types. For container types we make it
> a pointer when it has a default value, so we determine IsSet* by checking if
> the pointer is nil. But for string we make it not a pointer when it has
> default value, and determine IsSet* by checking if the value is the same as
> the default. We should make them consistent.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
