Hi Mikel, On Fri, Mar 2, 2012 at 2:41 PM, Mikel Astiz <mikel.astiz.oss@xxxxxxxxx> wrote: > From: Mikel Astiz <mikel.astiz@xxxxxxxxxxxx> > > Functions obc_session_get and obc_session_pull nearly share the same > code, so the later can be achieved by just calling the first one. > > The session api is not modified in this patch. > --- > client/session.c | 22 +--------------------- > 1 files changed, 1 insertions(+), 21 deletions(-) > > diff --git a/client/session.c b/client/session.c > index 6497d70..595f684 100644 > --- a/client/session.c > +++ b/client/session.c > @@ -1001,27 +1001,7 @@ int obc_session_pull(struct obc_session *session, > const char *type, const char *targetfile, > session_callback_t function, void *user_data) > { > - struct obc_transfer *transfer; > - const char *agent; > - > - if (session->obex == NULL) > - return -ENOTCONN; > - > - if (session->agent != NULL) > - agent = obc_agent_get_name(session->agent); > - else > - agent = NULL; > - > - transfer = obc_transfer_register(session->conn, session->obex, > - agent, > - targetfile, > - NULL, type, > - NULL); > - if (transfer == NULL) { > - return -EIO; > - } > - > - return session_request(session, transfer, session_prepare_get, > + return obc_session_get(session, type, NULL, targetfile, NULL, 0, > function, user_data); > } > > -- > 1.7.6.5 Sounds good, ack. -- Luiz Augusto von Dentz -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html