[Libva] [PATCH] Add callbacks for error and info messages.

[Sean V Kelley]

On Thu, 2016-10-27 at 15:55 +0000, Xiang, Haihao wrote:
> > -----Original Message-----
> > From: Libva [mailto:libva-bounces at lists.freedesktop.org] On Behalf
> > Of
> > Emmanuel Gil Peyrot
> > Sent: Tuesday, October 25, 2016 10:56 PM
> > To: Sean V Kelley <seanvk at posteo.de>
> > Cc: libva at lists.freedesktop.org
> > Subject: Re: [Libva] [PATCH] Add callbacks for error and info
> > messages.
> > 
> > On Mon, Oct 24, 2016 at 03:54:17PM -0700, Sean V Kelley wrote:
> > > On Wed, 2016-10-19 at 13:37 +0100, Emmanuel Gil Peyrot wrote:
> > > > This lets any application using libva choose the best way to
> > > > report
> > > > info and error messages to the user, for example graphical
> > > > application can open a popup on errors and write info messages
> > > > in
> > > > the toolbar.
> > > 
> > > 
> > > What version of Libva are you testing this with?
> > 
> > Hi, I wrote and tested that patch against libva master,
> > 3b7e4999950a04fabd42edbead8c2f24c6cdf3cf, corresponding to 0.39.4.
> 
> User might use the APIs in a multithread program however cannot get
> the expected result. 

Good point Haihao.  Due to multiple contexts, you may not see the
results you are expecting depending on the UI thread and backing
process.  Your applicaiton would need to keep track of the contexts so
that anything provided is meaningful to the user.

This is not an unusual situation and one we deal with on ChromeOS.

This is more of a caveat and I'm okay with merging it.

You should just be aware.

Applied, thanks.

Sean

> 
> > 
> > > 
> > > Sean
> > 
> > --
> > Emmanuel Gil Peyrot
> > _______________________________________________
> > Libva mailing list
> > Libva at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/libva
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: This is a digitally signed message part
URL: <https://lists.freedesktop.org/archives/libva/attachments/20161027/31b6b7d9/attachment.sig>