lucy-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [lucy-issues] [jira] [Commented] (CLOWNFISH-58) Refine Hash API
Date Wed, 05 Aug 2015 00:49:04 GMT

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

ASF GitHub Bot commented on CLOWNFISH-58:
-----------------------------------------

GitHub user rectang opened a pull request:

    https://github.com/apache/lucy-clownfish/pull/30

    CLOWNFISH-58 Replace Find_Key with Has_Key

    Remove the obsolete and complex Find_Key from Hash, add the simpler Has_Key.

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

    $ git pull https://github.com/rectang/lucy-clownfish CLOWNFISH-58-hash-has-key

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

    https://github.com/apache/lucy-clownfish/pull/30.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 #30
    
----
commit 2c02650ca7292e47390f4802e8b3454192cc6a6b
Author: Marvin Humphrey <marvin@rectangular.com>
Date:   2015-08-05T00:35:27Z

    Replace Hash's `Find_Key` with `Has_Key`.
    
    Hash needs some way to indicate whether a key is present but mapped to
    NULL -- hence, `Has_Key`.
    
    `Find_Key` was used for a specific purpose in Lucy but is now obsolete.

----


> Refine Hash API
> ---------------
>
>                 Key: CLOWNFISH-58
>                 URL: https://issues.apache.org/jira/browse/CLOWNFISH-58
>             Project: Apache Lucy-Clownfish
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Marvin Humphrey
>            Assignee: Marvin Humphrey
>             Fix For: 0.5.0
>
>
> After reviewing the Hash API ([http://s.apache.org/yzA]), we decided to make
> some refinements:
> *   Remove Find_Key.
> *   Add Has_Key.
> *   Use `size_t` for capacity and other internal variables.
> *   Make Hash final.
> *   Expose the following methods as public: Clear, Store, Fetch, Delete, Keys,
>     Values, Get_Size, Equals
> *   Implement Clone() with shallow semantics.
> *   CLOWNFISH-2, CLOWNFISH-7, CLOWNFISH-35, CLOWNFISH-36



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

Mime
View raw message