<div dir="ltr">Hi<br><div><br><div class="gmail_quote"><div dir="ltr">On Tue, Nov 1, 2016 at 1:20 PM Pavel Grunt <<a href="mailto:pgrunt@redhat.com">pgrunt@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, 2016-10-31 at 17:15 +0100, Victor Toso wrote:<br class="gmail_msg">
> Hi,<br class="gmail_msg">
><br class="gmail_msg">
> On Thu, Oct 27, 2016 at 04:07:15PM -0500, Jonathon Jongsma wrote:<br class="gmail_msg">
> > Seems OK to me. Looks like it was deprecated in Feb 2012.  Anybody<br class="gmail_msg">
> > else<br class="gmail_msg">
> > have objections? <br class="gmail_msg">
><br class="gmail_msg">
> None. I also think it is safe to remove it now.<br class="gmail_msg">
> Pushed all the acked patches.<br class="gmail_msg">
><br class="gmail_msg">
<br class="gmail_msg">
there should be a follow up to keep spice_audio_new in the map-file<br class="gmail_msg">
and spice-glib-sym-file on `make -C src update-symbol-files`<br class="gmail_msg"></blockquote><div><br></div><div>That, and update of documentation..<br></div><div>Imho it's not very beneficial to remove it from public header (break API), and keep the symbol. But if we remove it from exported symbols, we break ABI.<br><br></div><div>I would rather keep it for now, until we bump spice-gtk for API/ABI break. No rush to do that either.<br></div><br></div><div class="gmail_quote">I'll send a revert patch.<br></div></div></div><div dir="ltr">-- <br></div><div data-smartmail="gmail_signature"><div dir="ltr">Marc-André Lureau<br></div></div>