[prev in list] [next in list] [prev in thread] [next in thread] 

List:       koffice-devel
Subject:    Re: Help with koStore:open needed
From:       Nicolas Goutte <nicolasg () snafu ! de>
Date:       2002-12-30 18:24:53
[Download RAW message or body]

As far as I have understood your problematic number is only the store name,. 
So you could perhaps give it some prefix, for example the file extension or 
"picture":
png100002010000004000000040A4B9FAC7.png
picture100002010000004000000040A4B9FAC7.png

So everybody could be happy. The embedding could still be supported and for 
OO, you could still use identifiers.

(Other possible prefixes: 'object', 'id', 'OO'...)

Have a nice day/evening/night!

On Monday 30 December 2002 17:18, Percy Leonhardt wrote:
> On Sunday 29 December 2002 22:58, David Faure wrote:
> > On Sunday 29 December 2002 16:51, Percy Leonhardt wrote:
> > > +  QRegExp rx( "^\\d+$" );
> > > +  if ( rx.search( intern ) != -1 )
> >
> > Hmm, no, since the name could also be 0/1/2/pictures/picture0.png (see
> > lib/store/SPEC). We have a namespace collision here, obviously, which is
> > always very ugly.
>
> I am not convinced. ;-)
> Because the parts '0', '1' and '2' would still get the special handling
> needed and 'pictures' and 'picture0.png' will be kept as they are. And
> filenames like '100002010000004000000040A4B9FAC7.png' will be handled
> correct i.e. not as a part but as a simple file...
>
> Feeding ' 0/1/2/pictures/picture0.png' into koStore::open gives
>
> koffice (lib store): Opening for reading
> 'part0/part1/part2/pictures/picture0.png'
>
> with my patch and the following with the old solution:
>
> koffice (lib store): Opening for reading
> 'part0/part1/part2/pictures/picture0.png'
>
> So my solution would not break anything. But feeding
> 'Pictures/100002010000004000000040A4B9FAC7.png' into koStore::open gives
>
> koffice (lib store): Opening for reading
> 'Pictures/part100002010000004000000040A4B9FAC7.png/maindoc.xml'
>
> without my patch and with my patch
>
> koffice (lib store): Opening for reading
> 'Pictures/100002010000004000000040A4B9FAC7.png'
>
> which is correct.
>
> Sorry to bug you but I'm not sure if my little patch is really wrong. At
> least this would fix the problem for me and keep embedding working so
> Shaheed would be happy too.
>
> 	Percy :-)
> _______________________________________________
> koffice-devel mailing list
> koffice-devel@mail.kde.org
> http://mail.kde.org/mailman/listinfo/koffice-devel

_______________________________________________
koffice-devel mailing list
koffice-devel@mail.kde.org
http://mail.kde.org/mailman/listinfo/koffice-devel
[prev in list] [next in list] [prev in thread] [next in thread]