#3179 closed defect (fixed)

New warnings from Time Changes

Reported by: Joel Sherrill Owned by: Sebastian Huber
Priority: highest Milestone: 5.1
Component: posix Version: 5
Severity: blocker Keywords:
Cc: Blocked By:
Blocking:

Description

New warnings after picking up your recent commits. How are you checking for warnings?

../../../../../../rtems/c/src/../../cpukit/posix/src/pthreadattrdefault.c:58:5: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
     &_POSIX_Threads_Default_attributes.affinitysetpreallocated,
     ^
../../../../../../rtems/c/src/../../cpukit/posix/src/adjtime.c: In function 'adjtime':
../../../../../../rtems/c/src/../../cpukit/posix/src/adjtime.c:85:16: warning: passing argument 1 of '_TOD_Adjust' from incompatible pointer type [-Wincompatible-pointer-types]
   _TOD_Adjust( &delta_as_timestamp );
                ^
In file included from ../../../../../../rtems/c/src/../../cpukit/posix/src/adjtime.c:28:0:
../../cpukit/../../../erc32/lib/include/rtems/score/todimpl.h:287:6: note: expected 'const struct timespec *' but argument is of type 'Timestamp_Control * {aka long long int *}'
 void _TOD_Adjust(
      ^~~~~~~~~~~
sparc-rtems4.12-ar: `u' modifier ignored since `D' is the default (see `U')
sparc-rtems4.12-ar: `u' modifier ignored since `D' is the default (see `U')
sparc-rtems4.12-ar: `u' modifier ignored since `D' is the default (see `U')
sparc-rtems4.12-ar: `u' modifier ignored since `D' is the default (see `U')
../../../../../../rtems/c/src/../../cpukit/libcsupport/src/__times.c: In function '_times':
../../../../../../rtems/c/src/../../cpukit/libcsupport/src/__times.c:60:31: warning: passing argument 1 of '_TOD_Get_zero_based_uptime' from incompatible pointer type [-Wincompatible-pointer-types]
   _TOD_Get_zero_based_uptime( &binuptime );
                               ^
In file included from ../../../../../../rtems/c/src/../../cpukit/libcsupport/src/__times.c:35:0:
../../cpukit/../../../erc32/lib/include/rtems/score/todimpl.h:215:20: note: expected 'Timestamp_Control * {aka long long int *}' but argument is of type 'struct bintime *'
 static inline void _TOD_Get_zero_based_uptime(
                    ^~~~~~~~~~~~~~~~~~~~~~~~~~
../../../../../../rtems/c/src/../../cpukit/libcsupport/src/__times.c:71:55: warning: passing argument 2 of '_Thread_Get_CPU_time_used' from incompatible pointer type [-Wincompatible-pointer-types]
   _Thread_Get_CPU_time_used( _Thread_Get_executing(), &bin_cpu_time_used );
                                                       ^
In file included from ../../../../../../rtems/c/src/../../cpukit/libcsupport/src/__times.c:37:0:
../../cpukit/../../../erc32/lib/include/rtems/score/threadimpl.h:906:6: note: expected 'Timestamp_Control * {aka long long int *}' but argument is of type 'struct bintime *'
 void _Thread_Get_CPU_time_used(
      ^~~~~~~~~~~~~~~~~~~~~~~~~

Change History (4)

comment:1 Changed on Oct 12, 2017 at 1:24:09 AM by Joel Sherrill

Owner: set to Sebastian Huber
Status: newassigned

comment:2 Changed on Oct 12, 2017 at 5:20:28 AM by Sebastian Huber <sebastian.huber@…>

In 58500540/rtems:

posix: Fix const qualifier warning

Update #2514.
Update #3179.

comment:3 Changed on Oct 12, 2017 at 5:27:11 AM by Sebastian Huber <sebastian.huber@…>

Resolution: fixed
Status: assignedclosed

In 16db540a/rtems:

Use right time format in _times()

Update #2740.
Close #3179.

comment:4 Changed on Nov 9, 2017 at 6:27:14 AM by Sebastian Huber

Milestone: 4.12.05.1

Milestone renamed

Note: See TracTickets for help on using tickets.