Menu

#5953 Further cleanup/regularisation of Callback_wrapper classes and trampolines

Fixed
Enhancement
2020-05-07
2020-05-01
No

Further cleanup/regularisation of Callback_wrapper classes and trampolines

Contains two commits:

Callback_wrapper:: move trampolines and make_smob to Callbacks class

Make Callback_wrapper a class template

Templating Callback_wrapper on the wrapped argument list avoids a lot
of code repetition.

http://codereview.appspot.com/553980044

Discussion

  • Anonymous

    Anonymous - 2020-05-02
    • Description has changed:

    Diff:

    
    
    • Needs: -->
    • Patch: new --> review
     
  • Anonymous

    Anonymous - 2020-05-02

    Passes make, make check and a full make doc.

     
  • Anonymous

    Anonymous - 2020-05-04
    • Patch: review --> countdown
     
  • Anonymous

    Anonymous - 2020-05-04

    Patch on countdown for May 6th

     
  • Anonymous

    Anonymous - 2020-05-07
    • Patch: countdown --> push
     
  • Anonymous

    Anonymous - 2020-05-07

    Patch counted down - please push

     
  • Anonymous

    Anonymous - 2020-05-07
    • labels: --> Fixed_2_21_2
    • status: Started --> Fixed
    • Patch: push -->
     
  • Anonymous

    Anonymous - 2020-05-07
    Issue 5953/1: Make Callback_wrapper a class template
    author  David Kastrup <dak@gnu.org> 
        Wed, 29 Apr 2020 23:12:10 +0100 (00:12 +0200)
    committer   David Kastrup <dak@gnu.org> 
        Thu, 7 May 2020 10:28:49 +0100 (11:28 +0200)
    commit  6a95c1d34f7cb7e4fb67d2279a917c9ef3cfdd6b
    
    Issue 5953/2: Callback_wrapper:: move trampolines and make_smob to Callbacks class
    author  David Kastrup <dak@gnu.org> 
        Fri, 1 May 2020 20:12:10 +0100 (21:12 +0200)
    committer   David Kastrup <dak@gnu.org> 
        Thu, 7 May 2020 10:28:49 +0100 (11:28 +0200)
    commit  3c13d9d745f9983deac6d217169764b527469d6b