Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

overloaded-box protocol changes #22086

Closed
wants to merge 10 commits into from
Closed

overloaded-box protocol changes #22086

wants to merge 10 commits into from

Commits on Mar 12, 2015

  1. Replace exchange_malloc/exchange_free demo with something unrelat… 

    …ed to Box.

Precursor for landing overloaded-`box`, since that will decouple the
`box` syntax from the exchange heap (and in fact will eliminate the
use of the two aforementioned lang items).

Instead, the new demonstration program shows a definition of the
`str_eq` lang item. (We do not have that many procedural lang-items to
choose from, which is a good sign for our efforts to decouple the
compiler from the runtime!)

(This previously used a demo of `panic_bounds_check`, but a `str_eq`
demonstration is both easier to code and arguably a more interesting
aspect of the language to discuss.)

Fix unsafe.md example.
    @pnkfelix
    pnkfelix committed Mar 12, 2015
    Configuration menu
    Copy the full SHA
    02bbef0 View commit details
    Browse the repository at this point in the history

  2. Added Box<_> type annotations where necessary for overloaded-box.

    @pnkfelix
    pnkfelix committed Mar 12, 2015
    Configuration menu
    Copy the full SHA
    f2f09a1 View commit details
    Browse the repository at this point in the history

  3. Switched from box to Box::new where necessary for overloaded-box. 

    See also issue 22405, which tracks going back to `box <expr>` if
possible in the future.

Precursor for overloaded-`box` and placement-`in`; see Issue 22181.
    @pnkfelix
    pnkfelix committed Mar 12, 2015
    Configuration menu
    Copy the full SHA
    7645a2d View commit details
    Browse the repository at this point in the history

  4. parser: switch from box (<place>) <expr> form to `in <place> { <blo… 

    …ck> }`.

Note that `box <expr>` itself remains unchanged.
    @pnkfelix
    pnkfelix committed Mar 12, 2015
    Configuration menu
    Copy the full SHA
    cbf9303 View commit details
    Browse the repository at this point in the history

  5. prototype Placer protocol for unstable overloaded-box and placement-in.

    @pnkfelix
    pnkfelix committed Mar 12, 2015
    Configuration menu
    Copy the full SHA
    0be597d View commit details
    Browse the repository at this point in the history

  6. Add feature-gates for desugaring-based box and placement-in. 

    update test/compile-fail/feature-gate-box-expr.rs to reflect new feature gates.

Part of what lands with Issue 22181.
    @pnkfelix
    pnkfelix committed Mar 12, 2015
    Configuration menu
    Copy the full SHA
    467345b View commit details
    Browse the repository at this point in the history

  7. Accommodate error message explosion from box-desugaring in some tests. 

    See discussion on Issue 22231.
    @pnkfelix
    pnkfelix committed Mar 12, 2015
    Configuration menu
    Copy the full SHA
    4a7df22 View commit details
    Browse the repository at this point in the history

  8. Update tests for desugaring box and placement-in. 

    Namely:

* Update run-pass/new-box-syntax
* Fix doc-embedded test for `alloc::boxed` to reflect new syntax.
* Fix test/debuginfo/box.rs to reflect new syntax.

Part of what lands with Issue 22181.
    @pnkfelix
    pnkfelix committed Mar 12, 2015
    Configuration menu
    Copy the full SHA
    056e581 View commit details
    Browse the repository at this point in the history

  9. Issue 22450: Address desugaring-box problems in [pretty] run-pass tes… 

    …t suite.

Precursor for overloaded-`box` and placement-`in`; see Issue 22181.
    @pnkfelix
    pnkfelix committed Mar 12, 2015
    Configuration menu
    Copy the full SHA
    2d44712 View commit details
    Browse the repository at this point in the history

  10. Workaround issue 22462 by moving static value into its own module. 

    Precursor for overloaded-`box` and placement-`in`; see Issue 22181.
    @pnkfelix
    pnkfelix committed Mar 12, 2015
    Configuration menu
    Copy the full SHA
    a0e3c71 View commit details
    Browse the repository at this point in the history