You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficserver.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2016/11/04 18:27:58 UTC

[jira] [Work logged] (TS-1257) Replace Tcl_Hash* with lib/ts/Map

     [ https://issues.apache.org/jira/browse/TS-1257?focusedWorklogId=31649&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-31649 ]

ASF GitHub Bot logged work on TS-1257:
--------------------------------------

                Author: ASF GitHub Bot
            Created on: 04/Nov/16 18:27
            Start Date: 04/Nov/16 18:27
    Worklog Time Spent: 10m 
      Work Description: GitHub user persiaAziz opened a pull request:

    https://github.com/apache/trafficserver/pull/1197

    TS-1257_ssl_cipher_name_table: replace TCL_HashTable with unordered_map

    Replacement of InkHashTable with unordered_map. This PR  partially addresses TS-1257. I plan to gradually replace other instances of InkHashTable in ATS as well. 

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/persiaAziz/trafficserver TS-1257-ssl_cipher_name_table

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/trafficserver/pull/1197.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 #1197
    
----
commit ebe4270bebed8958bba0f136d93284a8d4fe98d3
Author: Persia Aziz <pe...@yahoo-inc.com>
Date:   2016-11-04T18:18:06Z

    TS-1257_ssl_cipher_name_table: replace TCL_HashTable with unordered_map

----


Issue Time Tracking
-------------------

            Worklog Id:     (was: 31649)
            Time Spent: 10m
    Remaining Estimate: 0h

> Replace Tcl_Hash* with lib/ts/Map
> ---------------------------------
>
>                 Key: TS-1257
>                 URL: https://issues.apache.org/jira/browse/TS-1257
>             Project: Traffic Server
>          Issue Type: Improvement
>          Components: Cleanup, Core
>            Reporter: Igor Galić
>            Assignee: Syeda Persia Aziz
>             Fix For: 7.1.0
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> We have our own implementation of maps that we use in most new code. We have already discussed looking into removing the old Tcl cruft by replacing it with {{Map}}, but have neither followed up on the ML nor Jira - or in the code ;)
> This ticket is a reminder that this task exists and wants to be done!
> As to whether we simply replace the Tcl Implementation underneath, or visibly to all developers, replace {{InkHashTable}} with {{Map}}, remains to be discussed/decided/evaluated.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)