Re: ftbench: last updates

Werner LEMBERG <[email protected]>
Newsgroups gmane.comp.fonts.freetype.devel
Message-ID <[email protected]>
Hello Ahmet,


sorry for the late reply, I'm travelling right now.

> -Made documentation and comment line (will continue).

Very nice!  Some minor nits, in case you have still time today:

* Please ensure that lines in the documentation are not longer than 78
  characters (`src/tools/ftbench/README`).  Ideally, `README` comes in
  Markdown format, but this is not important.

* Use two spaces after a full dot in documentation (this simplifies
  the search for the beginning of a sentence with some editors like
  Emacs).

* Use `autopep8 -a -a -a` to format `tohtml.py` and do some manual
  editing to also shorten the remaining overlong lines to
  78 characters per line if possible and useful.

* There are some indentation errors in `ftbench.c` (functions
  `test_render`, `test_embolden`, etc.).

* Please synchronize `ftbench.c` with the git version in the
  `freetype-devel` repository (which is a trivial change only).

> In this version of the code, i just didnt splitted iterations into
> chunks, got same results with it.

OK.  Too bad that we couldn't find the real cause for the large time
differences, but hey, this is a GSoC project :-)


  Werner
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.