some comments:
* we don't use tabs in examples. only spaces (4). we strictly follow the
PEAR standards.
* we also don't use tabs for indentation
* enchant_broker_describe() lacks documentation on some sections
* you aren't using the &return.success; entity. it could be used for
example in enchant_broker_free()
* enchant_dict_check() needs some attention
* there is a &example.outputs.similar; entity that should be used for
example in enchant_broker_list_dicts().
* strange example in enchant_dict_quick_check():
if (enchant_broker_dict_exists($r,$tag)) {
}
// nothing to do there?
there are also a couple of typos, but we'll squash them later.
I think thats all for now :)
Thanks,
Nuno
----- Original Message -----
Hello,
Can someone review the new doc for PECL Enchant before I commit or
simply commit it if it looks ok?
It passes the test and is generated correctly:
http://blog.thepimp.net/misc/zip/enchant_docbook.tgz
The docbook is available here (you need to add &url.enchant;[1] to
the globals.ent):
http://blog.thepimp.net/misc/zip/html/ref.enchant.html
Thanks :)
Cheers,
-- Pierre
[1] <!ENTITY url.enchant "http://www.abisource.com/projects/enchant/";>
