On Thu, Aug 16, 2007 at 03:57:40PM +0200, Abdelrazak Younes wrote: > Jean-Marc Lasgouttes wrote: > >Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes: > > > >>The following patch introduces LabelString for counters. This > >>labelstring will be used in several places: > > > >Here is the updated version of the patch, the shortcoming have been > >resolved and the layout files have been updated. For example floats > >and footnotes are now correctly numbered by chapter for book and > >report classes. > > > >I updated the documentation too. > > > >Note that the rest of the layut files should be updated. I do not > >think they are broken by the patch, but they can be simplified. > > Some cosmetics comments: > > (Counters::newCounter): add relevant arguments; allow to > redefine an existing counter. > > Why not Counters::new()? You already know are using the counters and are > going to get a new Counter.
A bit verbosity usually does not hurt, it's easier to grep fro, for once. And it has the advantage to compile in this particular case. Andre'
