lucy-issues mailing list archives

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


ASF GitHub Bot commented on CLOWNFISH-58:

GitHub user rectang opened a pull request:

    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 CLOWNFISH-58-hash-has-key

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

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 <>
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:
>             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 ([]), 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.

This message was sent by Atlassian JIRA

View raw message