[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