[Libreoffice] [PATCH] Base fdo#40079 "file / save (as)" inoperant

Noel Power nopower at novell.com
Thu Aug 18 06:06:54 PDT 2011


On 17/08/11 19:02, Lionel Elie Mamane wrote:
>
>>> Which leads me to another bug: If I remove the librarie's only dialog
>>> and save, I restart LO, I reopen the file again, the dialog is
>>> back.
>> probably worth opening another bug for that then, anyway, lets look
>> at one thing at a time ( otherwise I will get depressed )
> Fixed (by me) in master.
>
maybe  a better test would be

+ if ( ! ( xNameAccess->hasElements() || bInplaceStorage ) )

instead of

+ if ( ! ( xNameAccess->hasElements() || isModified() ) )

afaics in all cases where we have inplace storage we need to regenerate 
the basic related streams ( although I don't know where else other than 
base this is the case )  I just worry there might be some strange case 
where isModified might not be correct ( I have some vague recollection 
of it not being totally reliable under some circumstances...  )

regards,
Noel


More information about the LibreOffice mailing list