Re: [PATCH 1/1] grl-net: Use libsoup cache
- From: "Juan A." Suárez Romero <jasuarez igalia com>
- To: grilo-list gnome org
- Subject: Re: [PATCH 1/1] grl-net: Use libsoup cache
- Date: Tue, 15 Mar 2011 14:05:05 +0100
On Tue, 2011-03-15 at 13:48 +0100, Víctor M. Jáquez L. wrote:
> On Tue, Mar 15, 2011 at 01:27:14PM +0100, Juan A. Suarez Romero wrote:
>
> nitpick: I'm not big fan of function predefining. With out them, the code flow is more
> natural in my opinion.
>
> >
> > +#ifdef LIBSOUP_USE_UNSTABLE_REQUEST_API
> > +static void cache_down (GrlNetWc *self);
> > +static void cache_up (GrlNetWc *self);
> > +static SoupCache *get_cache();
> > +#endif
>
> And I guess we could get rid of these predefinitions, specially the get_cache(),
> which is only used by cache_down() and cache_up().
>
Uhm... yes, you're right. I'll change it.
Thanks
J.A.
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]