Bugzilla – Full Text Bug Listing
| Summary: | NPE when setting a function breakpoint | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [WebTools] JSDT | Reporter: | Simon Kaegi <simon_kaegi> | ||||
| Component: | Debug | Assignee: | Simon Kaegi <simon_kaegi> | ||||
| Status: | RESOLVED FIXED | QA Contact: | Simon Kaegi <simon_kaegi> | ||||
| Severity: | normal | ||||||
| Priority: | P3 | CC: | Michael_Rennie, thatnitind | ||||
| Version: | 3.2 | Flags: | Michael_Rennie:
review+
|
||||
| Target Milestone: | 3.2 RC1 | ||||||
| Hardware: | PC | ||||||
| OS: | Windows XP | ||||||
| Whiteboard: | |||||||
| Attachments: |
|
||||||
Created attachment 167430 [details]
proposed patch
+1 looks good Applied patch to HEAD |
I'm getting an NPE when setting a breakpoint on a function with no name. e.g. something like: AssertTestCase.prototype.testAssertFalse = function() { assertFalse(false); assertFalse("test", false); assertFalse(true, false); }; The problem is on line 494 of BreakpointLocationFinder this.functionName = node.getName().getIdentifier(); we should instead do something like: this.functionName = node.getName() == null ? null : node.getName().getIdentifier();