[Libreoffice] Help fixing ImplHandleInputLangChange implementation

Michael Meeks michael.meeks at novell.com
Mon Jun 20 12:27:30 PDT 2011


Hi Jesus,

On Mon, 2011-06-20 at 20:47 +0200, Jesús Corrius wrote:
> What's the preferred way to fix it?

	:-)

> a) put everything in an if (pFrame) {....} block
> b) if(!pFrame) return;

	It can help debugging if methods always return from the same place; but
then it can help code readability if it is not incredibly deeply
indented.

> Thanks for the help! :)

	Sorry for not helping :-) just do what you think fits best with that
module's practise and style.

	Thanks !

		Michael.;

-- 
 michael.meeks at novell.com  <><, Pseudo Engineer, itinerant idiot




More information about the LibreOffice mailing list