Re: [client v3 05/10] audio: Remove spice_audio_new() from the public header

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi

On Tue, Nov 1, 2016 at 1:20 PM Pavel Grunt <pgrunt@xxxxxxxxxx> wrote:
On Mon, 2016-10-31 at 17:15 +0100, Victor Toso wrote:
> Hi,
>
> On Thu, Oct 27, 2016 at 04:07:15PM -0500, Jonathon Jongsma wrote:
> > Seems OK to me. Looks like it was deprecated in Feb 2012.  Anybody
> > else
> > have objections? 
>
> None. I also think it is safe to remove it now.
> Pushed all the acked patches.
>

there should be a follow up to keep spice_audio_new in the map-file
and spice-glib-sym-file on `make -C src update-symbol-files`

That, and update of documentation..
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.

I would rather keep it for now, until we bump spice-gtk for API/ABI break. No rush to do that either.

I'll send a revert patch.
--
Marc-André Lureau
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]