2 function names different only by case in polyline.cxx (vcl outdev)

Jan Iversen jani at documentfoundation.org
Fri Sep 23 14:05:00 UTC 2016


> On 23 Sep 2016, at 14:16, julien2412 <serval2412 at yahoo.fr> wrote:
> 
> Hello,
> 
> I noticed 2 functions with the same name:
> void OutputDevice::DrawPolyLine( const tools::Polygon& rPoly, const
> LineInfo& rLineInfo )
> http://opengrok.libreoffice.org/xref/core/vcl/source/outdev/polyline.cxx#111
> 
> void OutputDevice::drawPolyLine(const tools::Polygon& rPoly, const LineInfo&
> rLineInfo) 239
> http://opengrok.libreoffice.org/xref/core/vcl/source/outdev/polyline.cxx#239
> 
> I suppose for case sensitive os, it shouldn't be a problem but for OS like
> Win? (or perhaps I missed something)
> 
> Any thoughts?
It does not seem to be a problem for the compilers, but it is surely a problem for any tired eye, I would highly recommend either combining the 2 or renaming one of them.

rgds
jan i.
> 
> Julien
> 
> 
> 
> --
> View this message in context: http://nabble.documentfoundation.org/2-function-names-different-only-by-case-in-polyline-cxx-vcl-outdev-tp4194590.html
> Sent from the Dev mailing list archive at Nabble.com.
> _______________________________________________
> LibreOffice mailing list
> LibreOffice at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/libreoffice



More information about the LibreOffice mailing list