The open() system call in f8 really broken...

Simo Sorce ssorce at redhat.com
Thu Aug 16 23:07:26 UTC 2007


On Thu, 2007-08-16 at 17:48 -0500, Eric Sandeen wrote:
> Roland McGrath wrote:
> >> One unfortunate side-effect of this is that cscope doesn't find these as
> >> "open" callers now.  Maybe I need a smarter cscope.  It gets it to
> >> build, though.
> > 
> > Yes, cscope should learn C.
> > 
> >>> retval = test_io_backing_manager->(open)(name, flags, &data->real);
> >> FWIW this didn't help, build failed same as before.
> > 
> > I'm not sure we believe you.
> 
> sorry, different failure.
> 
> test_io.c: In function ‘test_open’:
> test_io.c:198: error: expected identifier before ‘(’ token
> make[2]: *** [test_io.o] Error 1
> 
> bash-3.2# cat -n lib/ext2fs/test_io.c | grep 198
>    198                  retval = test_io_backing_manager->(open)(name,
> flags, &data->real);
> 
> *shrug* maybe I'm just dense and/or worn out today.  I'll go with what
> works.

Imo the "right" fix should be to not use "open" as I'd consider it a
reserved word, and change it to something like tio_open ... cscope would
be happy as well.

Simo.




More information about the Fedora-maintainers mailing list