[Ffmpeg-devel] BlackFin lowlevel pixel operations PATCH

Måns Rullgård mans
Wed Mar 28 13:26:01 CEST 2007


Marc Hoffman said:
> M?ns Rullg?rd writes:
>  >
>  > Marc Hoffman said:
>  > >
>  > > M?ns Rullg?rd writes:
>  > >  > Marc Hoffman <mmh at pleasantst.com> writes:
>  > >  >
>  > >  > > Not to ask silly questions what are _names reserved for?
>  > >  >
>  > >  > They are reserved by the C standard for use by the implementation,
>  > >  > i.e. compiler and libc.
>  > >  >
>  > >  > > And what do you mean by tabs are forbidden in svn?
>  > >  >
>  > >  > We don't allow tabs in our source code, and we've set up svn to reject
>  > >  > any checkin containing tabs.  Same for trailing whitespace.
>  > >
>  > > untabify -- done.. (to satisfy svn)
>  > > renamed _routine to z_routine (to maintain consistency)
>  >
>  > Are these functions externally visible?  Then they should have a ff_ prefix.
>  >
>  > > No problem what do we do about Makefiles and tabs?
>  >
>  > We have a special exception for makefiles.
>  >
>  > Two more requests: 1) please stop top-posting, and 2) trim signatures from
>  > replies.
>
> Is this better I guess I'm a bit unclear on the top post stuffs.

There's nothing to unclear about.  Just don't do it.  Ever.

> Thanks I will rename to ff_ actually all my routines will get that
> based on your suggestion.

Symbols with static linkage can be named anything you like outside the reserved
namespace (starting with _).  Anything with extern linkage needs a prefix to
avoid clashes with other libraries.

-- 
M?ns Rullg?rd
mans at mansr.com




More information about the ffmpeg-devel mailing list