On 12/05/14 12:32, Jeff Law wrote:1
fixincludes/ChangeLog:
    PR other/63613
    * inclhack.def (dejagnu_h_make_inline_functions_static): New fix.
    * fixincl.x: Regenerate.
    * tests/base/dejagnu.h: New.
OK.

No, actually not.

+fix = {
+    hackname = dejagnu_h_make_inline_functions_static;
+    files = dejagnu.h;
+
+    sed = 's@^inline void$'
+           '@static inline void@';

I guess I should elaborate on the preferred substitution mechanism: the 
"format" fix.
This is functionally equivalent but does not require loading the "sed" program:

fix = {
  hackname = dejagnu_static_inline;
  files = dejagnu.h;
  select = '^inline void$';
  c-fix = format;
  c-fix-arg = 'static %0';

Thank you.

Reply via email to