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

Ambiguity warnings on definition -> error on usage #16125

Merged
merged 5 commits into from
May 5, 2016
Merged

Ambiguity warnings on definition -> error on usage #16125

merged 5 commits into from
May 5, 2016

Commits on May 4, 2016

  1. Move ambiguity warnings from definition to usage: register ambiguities 

    For now, this disables any kind of ambiguity warning/error
    @timholy
    timholy committed May 4, 2016
    Configuration menu
    Copy the full SHA
    39bb06d View commit details
    Browse the repository at this point in the history

  2. Throw error for an ambiguous call

    @timholy
    timholy committed May 4, 2016
    Configuration menu
    Copy the full SHA
    827fb97 View commit details
    Browse the repository at this point in the history

  3. Refactor: methods returns multiples in case of ambiguity 

    Also:
- eliminates a lot of the dedicated error-handling for ambiguous methods in favor of returning NULL
- causes `precompile` to return true/false upon success/failure (useful for testing)
    @timholy
    timholy committed May 4, 2016
    Configuration menu
    Copy the full SHA
    122b5b4 View commit details
    Browse the repository at this point in the history

  4. Add tools for testing for ambiguities

    @timholy
    timholy committed May 4, 2016
    Configuration menu
    Copy the full SHA
    be8679e View commit details
    Browse the repository at this point in the history

  5. Add docs and NEWS

    @timholy
    timholy committed May 4, 2016
    Configuration menu
    Copy the full SHA
    76fa604 View commit details
    Browse the repository at this point in the history