You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@thrift.apache.org by "Bryan Duxbury (JIRA)" <ji...@apache.org> on 2009/03/13 16:13:50 UTC

[jira] Updated: (THRIFT-368) setFieldValue should allow nulls for all field types

     [ https://issues.apache.org/jira/browse/THRIFT-368?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Bryan Duxbury updated THRIFT-368:
---------------------------------

    Patch Info: [Patch Available]

> setFieldValue should allow nulls for all field types
> ----------------------------------------------------
>
>                 Key: THRIFT-368
>                 URL: https://issues.apache.org/jira/browse/THRIFT-368
>             Project: Thrift
>          Issue Type: Improvement
>          Components: Compiler (Java)
>            Reporter: Bryan Duxbury
>            Assignee: Bryan Duxbury
>            Priority: Trivial
>         Attachments: thrift-368.patch
>
>
> Right now, there's no generic field unsetter, and while you might think that setFieldValue(fid, null) would do the job, as soon as it runs into a primitive field, it will throw an NPE. I think it makes sense to check for null and if it's null, call the unsetter for that field.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.