Re: Full Screen Information
Carles Pina i Estany <[email protected]>
| Newsgroups | gmane.comp.kde.gwenview |
|---|---|
| Message-ID | <[email protected]> |
Hello, On Jan/19/2008, Aurélien Gâteau wrote: > Carles Pina i Estany a écrit : > > Hello! > > > > On Jan/15/2008, Aurélien Gâteau wrote: > > > > I send a new patch. But some help is welcomed in one part :-) (not fully > > working) > > I will have a look at it tomorrow I think. But I changed a few things in > the API which probably broke your patch: ImageMetaInfo is now known as > ImageMetaInfoModel. Should be easy to fix I think. Yes, it's now broken and I will try to fix today :-) > I also plan to make Document provide a pointer to an instance of > ImageMetaInfoModel instead of directly accessing the exiv2::Image > instance. This should simplify your code a bit more. I've looked on it just now. I guess that you did it yesterday :-) ------------------------------------------------------------------------ r766478 | gateau | 2008-01-25 23:51:43 +0100 (dv, 25 gen 2008) | 2 lines Document now provides an instance of ImageMetaInfoModel. ------------------------------------------------------------------------ Yeap, this will simplify my patch, I can use now getInfoForKey and so on from ImageMetaInfoModel. Actually, before (last weeks), I didn't realize that we could add this to Document to access ImageMetaInfoModel and then avoid working on Exiv2 in two places. Nice! -- Carles Pina i Estany GPG id: 0x8CBDAE64 http://pinux.info Manresa - Barcelona ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/