[parrot/parrot] d279ea: [GH #372] Removed the morph op by a deprecation st...

GitHub noreply at github.com
Wed Mar 5 22:40:06 UTC 2014


  Branch: refs/heads/ptc/rm_morph-gh372
  Home:   https://github.com/parrot/parrot
  Commit: d279ea9732837fd4959e5f503fadb527298ccb90
      https://github.com/parrot/parrot/commit/d279ea9732837fd4959e5f503fadb527298ccb90
  Author: Reini Urban <rurban at cpanel.net>
  Date:   2014-03-05 (Wed, 05 Mar 2014)

  Changed paths:
    M PBC_COMPAT
    M include/parrot/oplib/core_ops.h
    M include/parrot/oplib/ops.h
    M include/parrot/opsenum.h
    M src/ops/core_ops.c
    M src/ops/pmc.ops

  Log Message:
  -----------
  [GH #372] Removed the morph op by a deprecation stub

This is necessary so that we can remove the morph VTABLE as mentioned
in GH #372
Bumped PBC_COMPAT to 13.1


  Commit: 3e55f4b9a66ba1d0b43d92d319512a53c6add551
      https://github.com/parrot/parrot/commit/3e55f4b9a66ba1d0b43d92d319512a53c6add551
  Author: Paul Cochrane <paultcochrane at gmail.com>
  Date:   2014-03-05 (Wed, 05 Mar 2014)

  Changed paths:
    M t/oo/vtableoverride.t
    M t/pmc/undef.t

  Log Message:
  -----------
  [GH #372] Corrected tests using the 'morph' op

Now the test suite passes again :-)


  Commit: 37b5bd59f93da406b6b65530d650fd4db610722d
      https://github.com/parrot/parrot/commit/37b5bd59f93da406b6b65530d650fd4db610722d
  Author: Paul Cochrane <paultcochrane at gmail.com>
  Date:   2014-03-05 (Wed, 05 Mar 2014)

  Changed paths:
    M src/call/args.c
    M src/embed/api.c

  Log Message:
  -----------
  [GH #372] Replaced PMCNULL-setting morphs with the code directly

In src/call/args.c and src/embed/api.c VTABLE_morph() was just setting the
called object to PMCNULL.  The VTABLE_morph() has been replaced with a macro
MORPH_REPLACE_NULL which basically replicates the code in-place.  This
change should allow us to eventually remove the morph VTABLE entry.


  Commit: a403ba6eeed0064dd7dddd28c97681add557bf03
      https://github.com/parrot/parrot/commit/a403ba6eeed0064dd7dddd28c97681add557bf03
  Author: Paul Cochrane <paultcochrane at gmail.com>
  Date:   2014-03-05 (Wed, 05 Mar 2014)

  Changed paths:
    M src/pmc.c

  Log Message:
  -----------
  [GH #372] Replaced VTABLE_morph() in check_set_std_props() with direct assignment

Hopefully this does what was intended (this comment is also part of this
change).  The tests pass, so at least the probability that nothing is broken
is high, but unfortunately not 100%.


  Commit: ad4de7fb8828c62d7b8c48becceb71d7e7eb50b9
      https://github.com/parrot/parrot/commit/ad4de7fb8828c62d7b8c48becceb71d7e7eb50b9
  Author: Paul Cochrane <paultcochrane at gmail.com>
  Date:   2014-03-05 (Wed, 05 Mar 2014)

  Changed paths:
    M src/pmc/callcontext.pmc
    M src/pmc/default.pmc
    M src/pmc/object.pmc

  Log Message:
  -----------
  [GH #372] Removed VTABLE_morph implementations


  Commit: 65a40c1a862c2ec76ff45b1c65b519e7a4a01413
      https://github.com/parrot/parrot/commit/65a40c1a862c2ec76ff45b1c65b519e7a4a01413
  Author: Paul Cochrane <paultcochrane at gmail.com>
  Date:   2014-03-05 (Wed, 05 Mar 2014)

  Changed paths:
    M src/vtable.tbl

  Log Message:
  -----------
  [GH #372] Removed the morph() vtable entry


  Commit: e64e2fb1574b2733ef5d292b53d181760176de64
      https://github.com/parrot/parrot/commit/e64e2fb1574b2733ef5d292b53d181760176de64
  Author: Paul Cochrane <paultcochrane at gmail.com>
  Date:   2014-03-05 (Wed, 05 Mar 2014)

  Changed paths:
    M t/oo/vtableoverride.t
    M t/src/extend_vtable.t

  Log Message:
  -----------
  [GH #372] Removed tests of the morph() vtable after its removal


Compare: https://github.com/parrot/parrot/compare/d279ea973283^...e64e2fb1574b


More information about the parrot-commits mailing list