diff options
| author | Tor Lillqvist <tml@iki.fi> | 2013-03-11 11:47:25 +0200 |
|---|---|---|
| committer | Tor Lillqvist <tml@iki.fi> | 2013-03-11 12:14:44 +0200 |
| commit | 494f9304086de829b0ab372b35c337bc50236f33 (patch) | |
| tree | 1a32f5faf8e0420af29893baadafed14f6c1ce4c | |
| parent | 32e593dc2da4266567cf80ee6bea7917e56572c7 (diff) | |
Fix fallout from 3137258e346a9c16b2e61747fd926a1e7d358d02
WaE: expression which evaluates to zero treated as a null pointer
constant of type 'OutputDevice *'.
In this case, I think the code would have worked despite the scary
warning. (By luck, as the sal_False got passed as a NULL OutputDevice
pointer, and the following sal_Boolean parameter, which the sal_False
presumably was intended to mean, has the default value of sal_False.)
But better to make it more explicit which overload of Paint() is
intended.
But seriously, what good is supposed to come from the overloading of
the function name Paint() here? Would it really be so horribly
old-fashioned and C-like to use two different function names for the
slightly different use cases?
Change-Id: Ib397c8c1785320d3a9c0579a24f2c28ee4d22796
| -rw-r--r-- | editeng/source/editeng/impedit3.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/editeng/source/editeng/impedit3.cxx b/editeng/source/editeng/impedit3.cxx index dbc691047bab..fd88e88c9970 100644 --- a/editeng/source/editeng/impedit3.cxx +++ b/editeng/source/editeng/impedit3.cxx @@ -3722,7 +3722,7 @@ void ImpEditEngine::Paint( ImpEditView* pView, const Rectangle& rRec, OutputDevi DBG_ASSERT( bVDevValid, "VDef could not be enlarged!" ); if ( !bVDevValid ) { - Paint( pView, rRec, sal_False /* ohne VDev */ ); + Paint( pView, rRec, 0, sal_False /* ohne VDev */ ); return; } |