| Summary: | [TMF] Update the Event Model code | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | z_Archived | Reporter: | Francois Chouinard <fchouinard> | ||||||||||
| Component: | LinuxTools | Assignee: | Francois Chouinard <fchouinard> | ||||||||||
| Status: | CLOSED FIXED | QA Contact: | Francois Chouinard <fchouinard> | ||||||||||
| Severity: | normal | ||||||||||||
| Priority: | P3 | ||||||||||||
| Version: | unspecified | ||||||||||||
| Target Milestone: | --- | ||||||||||||
| Hardware: | PC | ||||||||||||
| OS: | Linux | ||||||||||||
| Whiteboard: | |||||||||||||
| Bug Depends on: | |||||||||||||
| Bug Blocks: | 301613 | ||||||||||||
| Attachments: |
|
||||||||||||
|
Description
Francois Chouinard
Created attachment 145578 [details]
Event model refresh
Synched the code with the current version of the model and adjusted the JUnits.
Hi Francois, I see you're using junit4 here. That's not going to work as both JUnit 3 and JUnit 4 can't exist in the Eclipse Testing Framework at the same time. See bug #153429 for more details. Is it possible to move to JUnit 3? Hi Andrew, No problem. /fc After the first internal delivery (and some comments from the mailing list...), some changes will be made to the event model: - The notion of TmfFormat will disappear; its responsibilities will be split between TmfType and TmfContent - TmfType will now provide the field IDs - TmfContent gets to implement the parsing on demand - TmfContent will also maintain a pointer to its parent event - TmfFields will maintain a pointer to its parent content Code and document will be updated accordingly. Created attachment 151266 [details]
TMF Event update + JUnits
Preliminary patch to investigate updated model (doc to follow).
Warning: breaks the LTTng components
Created attachment 151267 [details]
TMF Event Update + JUnits
Regenerated the patch to include an overlooked component.
Created attachment 151666 [details]
TmfEvent update (2)
Updated patch
Comment on attachment 151666 [details]
TmfEvent update (2)
Patch was committed a while ago...
This is pretty much "it" for Helios. |