albert-github
albert-github
> I think we should just remove it. Do you mean the RS / RS3 usage code from CGAL?
``` for (int i = 0; i < points.size()-1; i++) ``` shouldn't this be: ``` for (int i = 0; i < points.size()-2; i++) ``` as otherwise `points[i+1]` will be...
Yes, the `tm` is the problem as this is the, dummy / placeholder, name of the argument. One should either, in this case, use no arguments at all or use...
Yes like: ``` * @pre `CGAL::is_closed(tm)` * @pre \link CGAL::is_closed `CGAL::is_closed(tm)` \endlink ``` result:  or use a descriptive text, as the usage if `tm` (is a bit contradicting the...
No it is not a doxygen issue, it is so called usage. You wrote "We are in the scope of the explanation of the function and we have introduced `tm`...
Where did you see the link. Probably the reason is that in the example only the part `is_closed` is hyperlinked in the precondition the full word is attempted to link
I assume you mean here the links of the function calls in the main function. The links in the comment and the links in the code (example) are handled differently....
@westes thanks for the quick response. Difficult question had to do quite a bit of digging, as far as I can tell it comes from https://sourceforge.net/projects/winflexbison/files/ and has there the...
Wouldn't it be more consistent to replace all `NamedParametersOut...` by `NPOut...` in this package? Analogous for `NamedParametersIn...` rename to `NPIn...` and `NamedParameters...` to `NP...`.