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)