You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@lucy.apache.org by "Marvin Humphrey (JIRA)" <ji...@apache.org> on 2011/01/15 03:59:45 UTC

[lucy-issues] [jira] Updated: (LUCY-131) Improve argument conversion by allot_params()

     [ https://issues.apache.org/jira/browse/LUCY-131?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Marvin Humphrey updated LUCY-131:
---------------------------------

    Attachment: allot_params.patch

The "allot_params.patch" diff file has three logical parts:

    * Changes to XSBind_allot_params().
    * Changes to the Clownfish compiler which autogenerates calls to
      XSBind_allot_params()
    * Changes to hand-rolled code XS bindings which invoke
      XSBind_allot_params().

On OS X 10.6 and Perl 5.12.2, the size of the compiled object holding the Perl
bindings, Lucy.o, shrinks from 1337120 bytes to 1212940 -- a 9% savings.

> Improve argument conversion by allot_params()
> ---------------------------------------------
>
>                 Key: LUCY-131
>                 URL: https://issues.apache.org/jira/browse/LUCY-131
>             Project: Lucy
>          Issue Type: Improvement
>          Components: Clownfish, Perl bindings
>            Reporter: Marvin Humphrey
>            Assignee: Marvin Humphrey
>            Priority: Minor
>         Attachments: allot_params.patch
>
>
> The Perl bindings presently use too much code to convert Perl-space arguments
> to C-space variables.  We can improve matters by refactoring
> XSBind_allot_params() so that it assigns to C-space variables directly instead
> of parceling out Perl scalars for the caller to extract from, yielding both
> diminished visual clutter in the generated C code and a reduction in the size
> of the compiled object code.
> Dev list thread: [http://s.apache.org/MPA].

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.