You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficserver.apache.org by "Leif Hedstrom (JIRA)" <ji...@apache.org> on 2014/05/29 00:05:01 UTC

[jira] [Commented] (TS-1833) Deprecate TSMimeHdrFieldValueStringInsert() (and family)

    [ https://issues.apache.org/jira/browse/TS-1833?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14011678#comment-14011678 ] 

Leif Hedstrom commented on TS-1833:
-----------------------------------

I moved all this out to 6.0.0, but added a small comment to the include file.

> Deprecate TSMimeHdrFieldValueStringInsert() (and family)
> --------------------------------------------------------
>
>                 Key: TS-1833
>                 URL: https://issues.apache.org/jira/browse/TS-1833
>             Project: Traffic Server
>          Issue Type: Improvement
>          Components: TS API
>            Reporter: Leif Hedstrom
>            Assignee: Leif Hedstrom
>              Labels: api-change
>             Fix For: 6.0.0
>
>
> It seems to be that TSMimeHdrFieldValueStringInsert() is really just a subset of TSMimeHdrFieldValueStringSet(). This just makes for confusing APIs, i.e. which one do I use when? The alternative would be to remove the "idx" argument to the TSMimeHdrFieldValueStringSet() method, but that would then break API and ABI compatibility.
> Also, as James found out, the docs are less than clear. Set() needs to be called with an idx of -1 for it to actually be a Set() operation. With idx >=0, TSMimeHdrFieldValueStringSet() is actually identical to TSMimeHdrFieldValueStringInsert()....



--
This message was sent by Atlassian JIRA
(v6.2#6252)