[vlc-devel] commit: libvlc: Don't use enum directly, it's a bad pratice for ABI compat. (Pierre d'Herbemont )

[Rémi Denis-Courmont]

Le mercredi 10 juin 2009 21:01:24 Pierre d'Herbemont, vous avez écrit :
> 2009/6/10 Rémi Denis-Courmont <remi at remlab.net>:
> >>  Are you sure that this change does not break the ABI ?
> >> (I think there is other enum change like that one) ?
> >
> > That depends on the calling convention.
> > In other words, it does break the ABI.
>
> Yes, the change might break the ABI. Yet there might be no constance
> over compiler in this case, so no real solid ABI.
>
> The problem (as pointed) with directly using enum is that certain
> compiler option could end up shorten the enum at convenience.
>
> Laurent is right, if there are other occurrence of this kind of direct
> enum usage, we might want to get rid of those.

Look, either we agree that 1.0 is compatible with 0.9 and we revert this crap. 
Or we agree that it is not and the ABI number is bumped and we do backport all 
the fixes from 1.1. There is no being "somewhat" compatible.

I thought we agreed to go for the FIRST option.

-- 
Rémi Denis-Courmont
http://www.remlab.net/