Skip to content
This repository has been archived by the owner on Mar 25, 2021. It is now read-only.

Narrow error location of member-access #1964

Merged
merged 1 commit into from Jan 2, 2017
Merged

Narrow error location of member-access #1964

merged 1 commit into from Jan 2, 2017

Conversation

ajafff
Copy link
Contributor

@ajafff ajafff commented Jan 2, 2017

PR checklist

  • Addresses an existing issue: #0000
  • New feature, bugfix, or enhancement
    • Includes tests
  • Documentation update
  • Enable CircleCI for your fork (https://circleci.com/add-projects)

What changes did you make?

Error is reported from the start of the member until the end of its name instead of marking whole function bodies.

Is there anything you'd like reviewers to focus on?

(optional)

Error is reported from the start of the member until the end of its name instead of marking whole function bodies.
Copy link
Contributor

@nchen63 nchen63 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@nchen63 nchen63 merged commit 436e4a7 into palantir:master Jan 2, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants