i.e.

static ::sw::mark::DdeBookmark* const lcl_FindDdeBookmark(...)

On primitive types making something return a X* const is meaningless, you cannot
enforce that the variable it is assigned to will not be allowed take another
value, i.e.

::sw::mark::DdeBookmark* foo = lcl_FindDdeBookmark()

is still perfectly possible.

const ::sw::mark::DdeBookmark* lcl_FindDdeBookmark() is a totally different
thing to ensure that the memory pointed to by foo cannot be modified and in
general is what was intended, which is why the warning exists. In this case
though, that's not possible. So just the simple attached patch is needed to
silence the warning