Re: [E-devel] E SVN: urandom IN trunk/ephoto: . data/themes/default src/bin

[Cedric B. <ced...@fr...>]

On Tue, Sep 14, 2010 at 11:03 AM, Viktor Kojouharov
<vko...@gm...> wrote:
> tbh I'm not sure it should be there. it does add another dependency, which
> might not be desirable. and the actual exif code is pretty small, so it can
> easily be copied around for other apps to use.
> and the actual rotation is still a bit rough, so it's definitely not ready
> for elementary, if that should be its home.

Duplicating code for a small dependencies, that could be optional,
sounds like a wrong excuse to me. Maybe some one more involved than me
in elementary could give his opinion on this subject. But in my
opinion, it should be done at elementary level.

> speaking of which, how can I tell the edje part to refit itself once it has
> been rotated by a map? the rel1 and rel2 blocks don't seem to be enforced,
> or maybe edje thinks that the part's geometry hasn't changed.

Don't know much about edje map code. But sounds reasonable to think
that it only handle rel1 and rel2 before applying map.
-- 
Cedric BAIL