[PATCH v1] Port ndbm
Thu Jul 25 10:56:00 GMT 2019
On Thu, Jul 25, 2019, 2:26 PM Corinna Vinschen <email@example.com> wrote:
> On Jul 25 11:07, Vaibhav Gupta wrote:
> > On Thu, Jul 25, 2019, 1:48 AM Corinna Vinschen <firstname.lastname@example.org>
> > > On Jul 24 09:22, Corinna Vinschen wrote:
> > > > On Jul 10 14:25, Vaibhav Gupta wrote:
> > > > > ---
> > > > > newlib/libc/include/ndbm.h | 93 ++++++++++++++
> > > > > newlib/libc/search/Makefile.am | 1 +
> > > > > newlib/libc/search/ndbm.c | 217
> > > > > 3 files changed, 311 insertions(+)
> > > > > create mode 100644 newlib/libc/include/ndbm.h
> > > > > create mode 100644 newlib/libc/search/ndbm.c
> > > >
> > > > Pushed.
> > >
> > > However, where's dbm_forder? It's declared in ndbm.h but there's no
> > > such function in ndbm.c.
> > >
> > Both dbm_forder and dbm_dirinfo are wrapped inside __BSD_VISIBLE,
> > dbm_dirinfo is defined in ndbm.c but it will not be usable i guess.
> Do you mean dbm_dirfno? Why should it be unusable?
Yah sorry, dbm_dirfno(). By unusable i meant it will run on BSD systems.
Hence it won't create any problem for us.
> > I explored a bit, found that some features are for BSD systems and are
> > compatible with POSIX,
> > Moreover dbm_forder and dbm_dirinfo are not mentioned in open group page.
> > .
> > I didn't planned to modify the contents much when i ported
> > It from FreeBSD and hence they are present here. If required I can remove
> > them and send a new patch.
> Yes for dbm_forder, but dbm_dirfno doesn't look wrong, just BSDish,
> which is fine when guarded.
Ok then i will remove the declaration of dbm_forder and send a new patch.
> Corinna Vinschen
> Cygwin Maintainer
> Red Hat
More information about the Newlib