[lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-10-29 Thread Petri Savolainen
Added validation tests for ODP std C library API. Signed-off-by: Petri Savolainen --- configure.ac | 1 + platform/linux-generic/test/Makefile.am | 1 + test/validation/Makefile.am | 1 + test/validation/std_clib/.gitignore | 1 + test/validatio

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-10-29 Thread Bill Fischofer
On Thu, Oct 29, 2015 at 9:52 AM, Petri Savolainen < petri.savolai...@nokia.com> wrote: > Added validation tests for ODP std C library API. > > Signed-off-by: Petri Savolainen > Reviewed-and-Tested-by: Bill Fischofer > --- > configure.ac | 1 + > platform/linux-ge

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Savolainen, Petri (Nokia - FI/Espoo)
Ping. From: EXT Bill Fischofer [mailto:bill.fischo...@linaro.org] Sent: Thursday, October 29, 2015 5:27 PM To: Savolainen, Petri (Nokia - FI/Espoo) Cc: LNG ODP Mailman List Subject: Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests On Thu, Oct 29, 2015 at 9:52 AM

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Christophe Milard
- FI/Espoo) > *Cc:* LNG ODP Mailman List > *Subject:* Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added > validation tests > > > > > > > > On Thu, Oct 29, 2015 at 9:52 AM, Petri Savolainen < > petri.savolai...@nokia.com> wrote: > > Added vali

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Savolainen, Petri (Nokia - FI/Espoo)
, Petri (Nokia - FI/Espoo) Cc: EXT Maxim Uvarov; LNG ODP Mailman List Subject: Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests This is not a ODP module, is it? should/could it be one? I am not sure either what is the point of testing libC? If the question is to make sure

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Christophe Milard
> > > *From:* EXT Christophe Milard [mailto:christophe.mil...@linaro.org] > *Sent:* Thursday, November 05, 2015 12:02 PM > *To:* Savolainen, Petri (Nokia - FI/Espoo) > *Cc:* EXT Maxim Uvarov; LNG ODP Mailman List > > *Subject:* Re: [lng-odp] [API-NEXT PATCH] validation: std_clib

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Christophe Milard
XT Christophe Milard [mailto:christophe.mil...@linaro.org] >> *Sent:* Thursday, November 05, 2015 12:02 PM >> *To:* Savolainen, Petri (Nokia - FI/Espoo) >> *Cc:* EXT Maxim Uvarov; LNG ODP Mailman List >> >> *Subject:* Re: [lng-odp] [API-NEXT PATCH] validation: std_clib:

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Maxim Uvarov
Merged, Maxim. On 10/29/2015 18:26, Bill Fischofer wrote: On Thu, Oct 29, 2015 at 9:52 AM, Petri Savolainen mailto:petri.savolai...@nokia.com>> wrote: Added validation tests for ODP std C library API. Signed-off-by: Petri Savolainen mailto:petri.savolai...@nokia.com>> Reviewed-an

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Savolainen, Petri (Nokia - FI/Espoo)
12:12 PM To: Savolainen, Petri (Nokia - FI/Espoo) Cc: EXT Maxim Uvarov; LNG ODP Mailman List Subject: Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests but why should ODP tests test the C lib, as it is not part of ODP? If there is a good reason to do so (which I do not

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Savolainen, Petri (Nokia - FI/Espoo)
fischo...@linaro.org>] Sent: Thursday, October 29, 2015 5:27 PM To: Savolainen, Petri (Nokia - FI/Espoo) Cc: LNG ODP Mailman List Subject: Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests On Thu, Oct 29, 2015 at 9:52 AM, Petri Savolainen mailto:petri.savolai...@nokia

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Christophe Milard
ophe Milard [mailto:christophe.mil...@linaro.org] > *Sent:* Thursday, November 05, 2015 12:12 PM > > *To:* Savolainen, Petri (Nokia - FI/Espoo) > *Cc:* EXT Maxim Uvarov; LNG ODP Mailman List > *Subject:* Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added > validation tests > &g

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Christophe Milard
EXT Christophe Milard [mailto:christophe.mil...@linaro.org] > *Sent:* Thursday, November 05, 2015 12:15 PM > > *To:* Savolainen, Petri (Nokia - FI/Espoo) > *Cc:* EXT Maxim Uvarov; LNG ODP Mailman List > *Subject:* Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added > validat

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Maxim Uvarov
ay, November 05, 2015 12:15 PM *To:* Savolainen, Petri (Nokia - FI/Espoo) *Cc:* EXT Maxim Uvarov; LNG ODP Mailman List *Subject:* Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests Also, this patch does not respect the test naming convention rules.

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Christophe Milard
at 11:16, Savolainen, Petri (Nokia - FI/Espoo) < >> petri.savolai...@nokia.com <mailto:petri.savolai...@nokia.com>> wrote: >> >> What rules exactly? >> >> *From:*EXT Christophe Milard [mailto:christophe.mil...@linaro.org >> &

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Savolainen, Petri (Nokia - FI/Espoo)
] Sent: Thursday, November 05, 2015 12:28 PM To: Savolainen, Petri (Nokia - FI/Espoo) Cc: EXT Maxim Uvarov; LNG ODP Mailman List Subject: Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests /doc/implementers-guide/implementers-guide.html: 2.1.2. Module test and naming

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Christophe Milard
ailto:christophe.mil...@linaro.org] > *Sent:* Thursday, November 05, 2015 12:28 PM > > *To:* Savolainen, Petri (Nokia - FI/Espoo) > *Cc:* EXT Maxim Uvarov; LNG ODP Mailman List > *Subject:* Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added > validation tests > > > >

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Christophe Milard
etri >> >> >> >> >> >> >> >> *From:* EXT Christophe Milard [mailto:christophe.mil...@linaro.org] >> *Sent:* Thursday, November 05, 2015 12:28 PM >> >> *To:* Savolainen, Petri (Nokia - FI/Espoo) >> *Cc:* EXT Maxim Uvarov; LNG ODP

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Savolainen, Petri (Nokia - FI/Espoo)
; LNG ODP Mailman List Subject: Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests ok.then it is the right place in the structure. Are you fixing the names, Petri? On 5 November 2015 at 11:35, Savolainen, Petri (Nokia - FI/Espoo) mailto:petri.savolai...@nokia.com>>

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Savolainen, Petri (Nokia - FI/Espoo)
er 05, 2015 12:53 PM To: Savolainen, Petri (Nokia - FI/Espoo) Cc: EXT Maxim Uvarov; LNG ODP Mailman List Subject: Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests Also, I am not sure about why we need to wrap the libC functions in ODP, how much we should do it, and h

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Christophe Milard
t; *To:* Savolainen, Petri (Nokia - FI/Espoo) > *Cc:* EXT Maxim Uvarov; LNG ODP Mailman List > *Subject:* Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added > validation tests > > > > ok.then it is the right place in the structure. Are you fixing the names, > Petri? &

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Christophe Milard
nokia.com> wrote: > > /** > > * @defgroup odp_std_clib ODP STD CLIB > > * @details > > * ODP version of often used C library calls > > * @{ > > */ > > > > The module name is “std_clib”. I did try to use that as the module name. > You can find e.g. st