[PATCH 2/2] [cpukit/POSIX] Renamed POSIX signal functions to match style of the other POSIX related functions

Andreas Heinig andreas.heinig at cs.tu-dortmund.de
Wed Mar 20 06:43:57 UTC 2013


On Tuesday 19 March 2013 09:21:35 Joel Sherrill wrote:
> + Your tree is NOT up to date. It has ChangeLog files in it.
True,
> + You did a blind search and replace since it changed ChangeLog
> entries before your change was made.
That is correct. However, I accidentally added the ChangeLog to the patch

> Now to see if I can understand what you were trying to
> do specifically. Can you give a very simple summary with
> details. Like "renamed X to Y". "Changed include from X to
> Y after renaming file." etc.  The patch appears to be the
> result of a global search and replace and it would be
> easier to review if we knew what that was.

Ok, simply speaking: I had found a warning in the file 
rtems/cpukit/sapi/src/posixapi.c. Here, the compiler complained about nested 
external definition of _POSIX_signals_Manager_Initialization in function 
_POSIX_API_Initialize.

After fixing the warning and having a closer look on _POSIX_API_Initialize, I 
saw this:

 64   _POSIX_signals_Manager_Initialization();
             ^               ^
 65   _POSIX_Threads_Manager_initialization();
 66   _POSIX_Condition_variables_Manager_initialization();
 67   _POSIX_Key_Manager_initialization();
 68   _POSIX_Mutex_Manager_initialization();
 69   _POSIX_Message_queue_Manager_initialization();
 70   _POSIX_Semaphore_Manager_initialization();
 71   _POSIX_Timer_Manager_initialization();
 72   _POSIX_Barrier_Manager_initialization();
 73   _POSIX_RWLock_Manager_initialization();
 74   _POSIX_Spinlock_Manager_initialization();

Compared to all the other initialization routines, the Signal Manager has 
different capitalization. Hence, I changed 's'ignals to 'S'ignals and 
'I'nitialization to 'i'nitialization.

And due to the fact that the whole POSIX signal subsystem uses the "wrong" 
prefix _POSIX_signals_, I have made a global search and replace to change it 
to _POSIX_Signals_.

Here is the command:
find . ! -path "*/.svn*" ! -path "*/.git*" -type f \
-exec sed -e"s/_POSIX_signals/_POSIX_Signals/g" -i

Regards,
Andreas





More information about the devel mailing list