[PATCH v3 2/2] Add Testsuite for PSXINTTYPES01

Gedare Bloom gedare at rtems.org
Thu Jun 13 18:58:39 UTC 2019


On Thu, Jun 13, 2019, 12:57 PM Gedare Bloom <gedare at rtems.org> wrote:

> Hi Vaibhav,
>
>
>
> On Thu, Jun 13, 2019, 11:51 AM Vaibhav Gupta <vaibhavgupta40 at gmail.com>
> wrote:
>
>> Remove comments on variable declaration.
>> Remove licence from Doc
>> Correct the assert condition for wcstoimax for negative errange condition.
>> ---
>>
>
> Please squash (git-rebase) the commits to make one, new commit with only
> the necessary code. Send the new commit as -v2 patch. Thanks
>

Sorry, not -v2, you can use -v3 again for the squashed commit. Use -i flag
to git-rebase. Ask if you need further guidance


>  testsuites/psxtests/psxinttypes01/init.c      | 26 +++++++++---------
>>  .../psxtests/psxinttypes01/psxinttypes01.doc  | 27 -------------------
>>  2 files changed, 13 insertions(+), 40 deletions(-)
>>
>> diff --git a/testsuites/psxtests/psxinttypes01/init.c
>> b/testsuites/psxtests/psxinttypes01/init.c
>> index 8eafae11ac..94df8206c0 100644
>> --- a/testsuites/psxtests/psxinttypes01/init.c
>> +++ b/testsuites/psxtests/psxinttypes01/init.c
>> @@ -52,16 +52,16 @@ rtems_task Init(rtems_task_argument ignored)
>>    int       base            = 10;
>>    int       invalid_base    = 40;
>>
>> -  char     *nptr1_p         = "123abc";
>> //char* pointing to string with positive number
>> -  char            *nptr1_p_errange = "9999999999999999999999";
>>  //char* pointing to string with positive number that is out of range
>> -  char     *nptr1_n         = "-123abc";                       //char*
>> pointing to string with negative number
>> -  char            *nptr1_n_errange = "-9999999999999999999999";
>> //char* pointing to string with negative number that is out of range
>> +  char     *nptr1_p         = "123abc";
>> +  char            *nptr1_p_errange = "9999999999999999999999";
>> +  char     *nptr1_n         = "-123abc";
>> +  char            *nptr1_n_errange = "-9999999999999999999999";
>>    char     *endptr1         = NULL;
>>
>> -  wchar_t  *nptr2_p         = L"123junk";
>> //wchar_t* pointing to string with positive number
>> -  wchar_t  *nptr2_p_errange = L"9999999999999999999999";
>>  //wchar_t* pointing to string with positive number that is out of range
>> -  wchar_t  *nptr2_n         = L"-123junk";
>>  //wchar_t* pointing to string with negative number
>> -  wchar_t  *nptr2_n_errange = L"-9999999999999999999999";
>> //wchar_t* pointing to string with negative number that is out of range
>> +  wchar_t  *nptr2_p         = L"123junk";
>> +  wchar_t  *nptr2_p_errange = L"9999999999999999999999";
>> +  wchar_t  *nptr2_n         = L"-123junk";
>> +  wchar_t  *nptr2_n_errange = L"-9999999999999999999999";
>>    wchar_t  *endptr2         = NULL;
>>
>>    intmax_t  result_strtoimax;
>> @@ -69,7 +69,7 @@ rtems_task Init(rtems_task_argument ignored)
>>
>>    TEST_BEGIN();
>>
>> -  /*Test for strtoimax  */
>> +  /* Test for strtoimax  */
>>    puts( "\nstrtoimax Testcases...." );
>>    puts( "Valid Inputs - Positive Number" );
>>    result_strtoimax = strtoimax( nptr1_p, &endptr1, base );
>> @@ -105,7 +105,7 @@ rtems_task Init(rtems_task_argument ignored)
>>    rtems_test_assert( result_strtoimax == 0 );
>>    rtems_test_assert( errno == EINVAL );
>>
>> -  /*Test for strtoumax  */
>> +  /* Test for strtoumax  */
>>    puts( "\nstrtoumax Testcases...." );
>>    puts( "Valid Inputs - Positive Number" );
>>    result_strtoumax = strtoumax( nptr1_p, &endptr1, base );
>> @@ -141,7 +141,7 @@ rtems_task Init(rtems_task_argument ignored)
>>    rtems_test_assert( result_strtoumax == 0 );
>>    rtems_test_assert( errno == EINVAL );
>>
>> -  /*Test for wcstoimax  */
>> +  /* Test for wcstoimax  */
>>    puts( "\nwcstoimax Testcases...." );
>>    puts( "Valid Inputs - Positive Number" );
>>    result_strtoimax = wcstoimax( nptr2_p, &endptr2, base );
>> @@ -164,7 +164,7 @@ rtems_task Init(rtems_task_argument ignored)
>>
>>    puts( "Valid Input - Negative Number - Number out of Range" );
>>    result_strtoimax = wcstoimax( nptr2_n_errange, &endptr2, base );
>> -  rtems_test_assert( result_strtoimax != 0 );
>> +  rtems_test_assert( result_strtoimax == INTMAX_MIN );
>>    rtems_test_assert( errno == ERANGE );
>>
>>    puts( "Invalid Input - Send NULL Pointer" );
>> @@ -177,7 +177,7 @@ rtems_task Init(rtems_task_argument ignored)
>>    rtems_test_assert( result_strtoimax == 0 );
>>    rtems_test_assert( errno == EINVAL );
>>
>> -  /*Test for wcstoumax  */
>> +  /* Test for wcstoumax  */
>>    puts( "\nwcstoumax Testcases...." );
>>    puts( "Valid Inputs - Positive Number" );
>>    result_strtoumax = wcstoumax( nptr2_p, &endptr2, base );
>> diff --git a/testsuites/psxtests/psxinttypes01/psxinttypes01.doc
>> b/testsuites/psxtests/psxinttypes01/psxinttypes01.doc
>> index b5fd8e6d0e..607d67015e 100644
>> --- a/testsuites/psxtests/psxinttypes01/psxinttypes01.doc
>> +++ b/testsuites/psxtests/psxinttypes01/psxinttypes01.doc
>> @@ -1,30 +1,3 @@
>> -/*
>> - * SPDX-License-Identifier: BSD-2-Clause
>> - *
>> - * Copyright (C) 2019, Vaibhav Gupta
>> - *
>> - * Redistribution and use in source and binary forms, with or without
>> - * modification, are permitted provided that the following conditions
>> - * are met:
>> - * 1. Redistributions of source code must retain the above copyright
>> - *    notice, this list of conditions and the following disclaimer.
>> - * 2. Redistributions in binary form must reproduce the above copyright
>> - *    notice, this list of conditions and the following disclaimer in the
>> - *    documentation and/or other materials provided with the
>> distribution.
>> - *
>> - * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
>> "AS IS"
>> - * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
>> THE
>> - * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
>> PURPOSE
>> - * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS
>> BE
>> - * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
>> - * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
>> - * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR
>> BUSINESS
>> - * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER
>> IN
>> - * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR
>> OTHERWISE)
>> - * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED
>> OF THE
>> - * POSSIBILITY OF SUCH DAMAGE.
>> - */
>> -
>>  This File describes the concepts tested by this test suite.
>>
>>  inttypes.h - fixed size integer types
>> --
>> 2.21.0
>>
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/devel/attachments/20190613/6bcc731d/attachment-0002.html>


More information about the devel mailing list