Re: [RFC spice-server 1/3] stream-channel: Add preferred video codec capability

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

 



Hi,

On Wed, Aug 14, 2019 at 09:08:51AM -0400, Frediano Ziglio wrote:
> > +/* Returns an array with SPICE_VIDEO_CODEC_TYPE_ENUM_END elements,
> > + * with the client preference order (index) as value */
> > +GArray
> > *video_stream_parse_preferred_codecs(SpiceMsgcDisplayPreferredVideoCodecType
> > *msg)
> > +{
> > +    gint i, len;
> 
> gnot ga gbig gfun gof gall gthese gg gstrings

Haha, you brought something like that in the past too. What can
we do about it? I don't even think much, if I'm working on code
with glib/gobject I g-prefix types too. I would say it makes
sense when interacting with the its libraries, random e.g:

    glong g_utf8_strlen (const gchar *p, gssize max);

I'm raising this question/email so we can improve this somehow
(checkpatch.pl git-hooks and/or coding style standards) and make
it clear for contributors/reviewers without much time loss in
discussion.

Cheers,

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel

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