Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [tracecompass-dev] Deprecating TimeGraphCombo


On 2016-12-15 06:27 PM, Patrick Tasse wrote:
On Thu, Dec 15, 2016 at 5:49 PM, Jonathan Rajotte Julien <Jonathan.rajotte-julien@xxxxxxxxxxxx <mailto:Jonathan.rajotte-julien@xxxxxxxxxxxx>> wrote:

    Could you provide a list of code needing that update? This would
    allow a faster and more complete change.

A subclass of AbstractTimeGraphView that doesn't use getTimeGraphCombo() should work out of the box, whether it uses columns or not.

In most cases calls to getTimeGraphCombo() can get replaced with getTimeGraphViewer().

Calls to setSelection(entry) on the combo should be replaced with setSelection(entry, true) on the time graph viewer.

If the user called getTree() on the combo, the tree returned by the time graph viewer only has column headers and no tree items. For example CallStackView was adding an IDoubleClickListener on the tree. This had to be replaced by a mouseDoubleClick() MouseListener on the time graph control, with the selection fetched from the viewer instead of from the DoubleClickEvent. That was the hardest update required in Trace Compass.

Thanks for the thorough "how to" but you are not answering the question.
You seem to imply that some code might need changes to prevent breaking.

"I can help anyone that needs advice for updating their code to use a time graph viewer instead of a time graph combo."

Thus the question "Could you provide a list of code needing that update?" regarding TC code base.

Let me rephrase: Does merging the proposed changes "as is" break stuff currently present in Trace Compass (also known as "regression")?

If so, since you are proposing the change and that you are a part of the maintaining team, I would expect that you provide a detailed list of what need to be done and by who (maintainers or specific contributors) to ensure that weird stuff do not happen and nothing break.
This would allow people like me, who would like this merged asap, to help.
You are the one currently playing in this part and have probably much more insight regarding what will break (you did not provide all of this information) and how it will break (you provided this information).

Otherwise, discard the previous paragraph (well you can still read it to make sure I do not say anything stupid).



tracecompass-dev mailing list
To change your delivery options, retrieve your password, or unsubscribe from this list, visit

Jonathan R. Julien

Back to the top