[PUSHED] Replacing '__LOADONCALLAPI' with 'SAL_CALL' definition

Michael Meeks michael.meeks at suse.com
Mon Dec 10 13:39:12 PST 2012


Hi there,

	Thanks for the updated patch :-)

On Mon, 2012-12-10 at 08:08 -0800, Christos Strubulis wrote:
> I haven't deleted helpcontent2 yet but I will do it before the
> next autogen and compilation. 

	I removed it from the patch and pushed.

> Would you like to have a look at the attached patch? What is
> your opinion now?

	Lubos asked you to:

> > - SAL_CALL is already defined in sal/inc/sal/types.h , so there's no need to 
> > define it again in tools/inc/tools/solar.h , the definition of 
> > __LOADONCALLAPI should be simply removed there
> > 
> >  Can you submit a new patch with these changes ?

	I did that for you, since your patch still had the definition of
SAL_CALL included - did you send the wrong patch somehow ?

	Anyhow - thanks for your first contribution; good to get it in.

	All the best,

		Michael.

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



More information about the LibreOffice mailing list