Bugzilla – Full Text Bug Listing
| Summary: | Array values are not shown | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [WebTools] JSDT | Reporter: | Michael Schneider <michschn> | ||||||||
| 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: | deboer:
pmc_approved+
Michael_Rennie: review+ |
||||||||
| Target Milestone: | 3.2 RC2 | ||||||||||
| Hardware: | PC | ||||||||||
| OS: | Windows 7 | ||||||||||
| Whiteboard: | |||||||||||
| Attachments: |
|
||||||||||
This is a regression and a must fix for RC2. Created attachment 168730 [details]
proposed patch
The change to add non-enumerated properties inadvertently removed any specialization of getIds done in both Arrays and Functions. We need to merge the two id sets to get all of the appropriate values.
patch could be a bit speedier if we just copy the values in to the set ourselves instead of using Arrays.asList and creating a garbage ArrayList... Created attachment 168766 [details]
screen shot
I also found that the details for an array object are including values that are not actually part of the Array. The screenshot shows the length of the array is being included as a value in the array (according to the var view and the details anyways)
Bug 313011 fixes up the variables view, this bug just makes sure we have the appropriate properties in array objects. works as intended, Simon promised to fix the details pane to once again show the array values and not just 'Array' before he commits it. Fixed in HEAD woops noticed I set a pmc_approved flag, fixing to set a review flag from me. |
Created attachment 168631 [details] Screenshot of the issue See attached screenshot I would expect that I can see the array entries as properties.