Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 315427 - Modify TRCObject class label and handle null values for sorting comparison
Summary: Modify TRCObject class label and handle null values for sorting comparison
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: TPTP (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P2 normal (vote)
Target Milestone: ---   Edit
Assignee: Joel Cayne CLA
QA Contact: Kathy Chan CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-02 12:59 EDT by Joel Cayne CLA
Modified: 2016-05-05 10:58 EDT (History)
3 users (show)

See Also:
kathy: pmc_approved? (oec)
kathy: pmc_approved? (ernest)
kathy: pmc_approved? (kathy)
kathy: pmc_approved? (paulslau)
jgwest: pmc_approved+
jgwest: review+


Attachments
Patch to update Label ID and sorting value (2.99 KB, patch)
2010-06-02 12:59 EDT, Joel Cayne CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Joel Cayne CLA 2010-06-02 12:59:39 EDT
Created attachment 170838 [details]
Patch to update Label ID and sorting value

The class label provided in the Analysis view for TRCObject is currently 'Class Name'.'TPTP Object ID'. This could be updated to be similar to the Debug view format of 'Class Name' (id='TPTP Object ID').

When sorting on an entry for a column that has no value and returns null, a null pointer exception is thrown. Instead of throwing the exception, if the value for the row in the sorted column is null a default value of 0 could be returned for comparison.
Comment 1 Joel Cayne CLA 2010-06-02 13:00:08 EDT
Jonathan,
Can you please review?

Thanks!
Comment 2 Jonathan West CLA 2010-06-02 13:02:38 EDT
Patch is good!
Comment 3 Joel Cayne CLA 2010-06-02 13:19:34 EDT
I would like to request PMC approval to include the following fix in TPTP 4.7.0.

1.   Explain why you believe this is a stop-ship defect. How does the defect
manifest itself, and how will users of TPTP / consuming products be affected if
the defect is not fixed?

Updates a label for TRCObject in the analysis view and fixes an NPE thrown when a row does not provide a value to sort on.

2. Is there a work-around? If so, why do you believe the work-around is
insufficient?

No.

3. Is this a regression or API breakage? Explain.

No.

4. Does this require new API?

No.

5. Who performed the code review?

Jonathan

6. Is there a test case attached to the bugzilla record?

n/a

7. What is the nature of the fix? What is the scope of the fix? What is the
risk associated with this fix?

Low - changing a label and returning a result to compare against 0 instead of receiving a null pointer exception.

8. Is this fix related to any standards that TPTP adheres to? If so, who has
validated that the fix continues to adhere to the standard?
n/a
Comment 4 Kathy Chan CLA 2010-06-02 13:22:29 EDT
Requesting PMC approval for TPTP 4.7.
Comment 5 Joel Cayne CLA 2010-06-02 17:06:50 EDT
Patch checked into HEAD.
Comment 6 Joel Cayne CLA 2010-06-11 12:16:55 EDT
Closing.