Bug 831677 - Add dict_set_transient_str
Summary: Add dict_set_transient_str
Keywords:
Status: CLOSED DEFERRED
Alias: None
Product: GlusterFS
Classification: Community
Component: core
Version: 3.3-beta
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ---
Assignee: Jeff Darcy
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-06-13 14:44 UTC by Jeff Darcy
Modified: 2014-12-14 19:40 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2014-12-14 19:40:28 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Jeff Darcy 2012-06-13 14:44:44 UTC
While reviewing http://review.gluster.com/#change,3565 I noticed that we have nearly a hundred uses of dict_set_dynstr, and a majority of those use gf_strdup'ed values.  This requires callers to check two return values, from gf_strdup and dict_set_dynstr, complicating error recovery.  I'd like to add a dict_set_transient_str function that does the gf_strdup internally, simplifying callers which then only need to check one return value.

Comment 1 Niels de Vos 2014-11-27 14:53:42 UTC
The version that this bug has been reported against, does not get any updates from the Gluster Community anymore. Please verify if this report is still valid against a current (3.4, 3.5 or 3.6) release and update the version, or close this bug.

If there has been no update before 9 December 2014, this bug will get automatocally closed.


Note You need to log in before you can comment on or make changes to this bug.