Changing feedback messages to be appended

[Durbatuluk1701]

This PR changes the behavior of handlings feedback messages to append the message rather than prepend them.
 
This should specifically fix #873

[Durbatuluk1701]

One thing I also noticed while debugging this was that it seems to me like debug auto gets broken up into many different messages and events. Maybe at some point we could explore a more efficient solution of processing one large event/message since all the currently generated events/messages apply to the same range and have the same severity?

[rtetley]

Thanks !

[rtetley]

One thing I also noticed while debugging this was that it seems to me like debug auto gets broken up into many different messages and events. Maybe at some point we could explore a more efficient solution of processing one large event/message since all the currently generated events/messages apply to the same range and have the same severity?

I agree this could be a good idea. Could you write it up in an issue to track this ?

[gares]

I suspect that for long output, like auto traces, it is better to still prepend but rev when displaying (less allocations, better complexity)

[Durbatuluk1701]

I suspect that for long output, like auto traces, it is better to still prepend but rev when displaying (less allocations, better complexity)

I was a bit worried that if any other tools ever tried to interact with vscoqtop they might be confused unless they also reversed feedback messages client side.
If you think it is worth it though I can definitely try the prepend and rev client-side approach.
Also might be worth considering if the append price is too high if #878 works out.