Re: [freetype2] priyesh_LogLibrary 92847d8: * builds/windows/ftdebug.c (ft_log_handler): Updated code according to the

[Werner LEMBERG]

Hello Priyesh,


> +    if( ft_getenv( "FT_LOGGING_FILE" ) )
> +    {
> +      ft_fileptr = fopen( ft_getenv( "FT_LOGGING_FILE" ) , "w" );
> +    }    
> +    else 
> +    {
> +      ft_fileptr = stderr;
> +    }

Minor issue: The coding style in FreeType doesn't use braces for
conditionals if there is only a single line.  In other words, code
like the above should rather be

  if ( ft_getenv( "FT_LOGGING_FILE" ) )
    ft_fileptr = fopen( ft_getenv( "FT_LOGGING_FILE" ), "w" );
  else 
    ft_fileptr = stderr;


     Werner