de-indent, given the early return

Change-Id: I7f52cad90ce17813ba7d7c6ce382d58a8fa44916
coverity#440865 Logically dead code

seeing as we return early if eChild == SFX_CHILDWIN_OBJECTBAR
then the reset of the code that depends on that cannot happen

Change-Id: I1566ad9d5ef428e3c9e248df08b65d56bb83553e
coverity#441017 Dereference before null check

Change-Id: I3df33251d72266afb43532bee07058e691a6cf66
coverity#441225 Dereference null return value

Change-Id: I9ff48e869cd03847d5338c5f8fb6acceee522e34
coverity#704232 Logically dead code

these conditions are already handled in the switch above

Change-Id: I1a2bfe4cc536f2d1292fd39b86afc53ac5463308
coverity#704236 Logically dead code

the nStartPos > 0 check is inside a nStartPos == 0 block
so it can't happen so we'll return immediately, and so
there's no point changing numberOfChange in that case
either there

Change-Id: Id6d7fc31033de5c6f3bb0a9ed03234b58c0d509e