[PATCH] lib: remove unused function nttime_from_string()

Jeremy Allison jra at samba.org
Fri Jan 11 17:45:47 UTC 2019


On Fri, Jan 11, 2019 at 10:06:25AM +0100, Volker Lendecke via samba-technical wrote:
> On Fri, Jan 11, 2019 at 09:51:44AM +0100, swen wrote:
> > On Fri, 2019-01-11 at 09:31 +0100, Volker Lendecke via samba-technical
> > wrote:
> > > On Fri, Jan 11, 2019 at 09:28:22AM +0100, swen via samba-technical
> > > wrote:
> > > > Please review and push if happy.
> > > 
> > > Can you also remove the prototype in lib/util/time.h?
> > Oh man, of course.
> > Sorry for that.
> > 
> > Attached patch updated.
> 
> RB+. Another one?

LGTM. RB+ and pushed !

Thanks,

Jeremy.

> > 
> > Cheers Swen
> 
> > From d1e2a89e262f484f9e144d2d9b608aca6c1d2061 Mon Sep 17 00:00:00 2001
> > From: Swen Schillig <swen at linux.ibm.com>
> > Date: Fri, 11 Jan 2019 09:26:29 +0100
> > Subject: [PATCH] lib: remove unused function nttime_from_string()
> > 
> > Signed-off-by: Swen Schillig <swen at linux.ibm.com>
> > ---
> >  lib/util/time.h    | 5 -----
> >  source3/lib/time.c | 9 ---------
> >  2 files changed, 14 deletions(-)
> > 
> > diff --git a/lib/util/time.h b/lib/util/time.h
> > index 9b897eb4b6c..1988b330576 100644
> > --- a/lib/util/time.h
> > +++ b/lib/util/time.h
> > @@ -180,11 +180,6 @@ void push_nttime(uint8_t *base, uint16_t offset, NTTIME t);
> >  */
> >  NTTIME pull_nttime(uint8_t *base, uint16_t offset);
> >  
> > -/**
> > -  parse a nttime as a large integer in a string and return a NTTIME
> > -*/
> > -NTTIME nttime_from_string(const char *s);
> > -
> >  /**
> >    return (tv1 - tv2) in microseconds
> >  */
> > diff --git a/source3/lib/time.c b/source3/lib/time.c
> > index 30ad1ec9a01..6b864253a91 100644
> > --- a/source3/lib/time.c
> > +++ b/source3/lib/time.c
> > @@ -36,15 +36,6 @@
> >  #define TIME_FIXUP_CONSTANT_INT 11644473600LL
> >  #endif
> >  
> > -
> > -/**
> > -  parse a nttime as a large integer in a string and return a NTTIME
> > -*/
> > -NTTIME nttime_from_string(const char *s)
> > -{
> > -	return strtoull(s, NULL, 0);
> > -}
> > -
> >  /**************************************************************
> >   Handle conversions between time_t and uint32, taking care to
> >   preserve the "special" values.
> > -- 
> > 2.20.1
> > 
> 
> 
> 
> 
> -- 
> SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
> phone: 0551-370000-0, mailto:kontakt at sernet.de
> Gesch.F.: Dr. Johannes Loxen und Reinhild Jung
> AG Göttingen: HR-B 2816 - http://www.sernet.de
> 



More information about the samba-technical mailing list