[vlc-devel] Fullscreen mouse hidding

Pierre d'Herbemont pdherbemont at free.fr
Sat Mar 22 22:28:57 CET 2008


On Mar 20, 2008, at 2:49 PM, Rafaël Carré wrote:
> On Thu, 2008-03-20 at 10:27 +0100, Lukas Durfina wrote:
>> Hi,
>>
>>   according to your comments I make a new patch.
>> I hope, it is fine for everyone. If not, give me
>> feedback, please.
>
> It is good.
>
> However I have a few remarks about coding style only:
> * You should prefix variables with a letter indicating their type (i_
> for integer, f_ for floats)
> * When adding new comments you should use C comments and not C++ /* */
>
> See http://wiki.videolan.org/Code_Conventions
>
> Once you send the patch with these, it can be pushed to master branch.

See also this:

http://wiki.videolan.org/Sending_Patches

On how to send patches using git.

Pierre.


More information about the vlc-devel mailing list