Re: [PATCH] cdev.h needs struct inode; add forward declaration




include/linux/cdev.h defines cd_forget to take a struct inode *, but does not
pull in any definition or declaration for struct inode. This generates a
compiler warning if a source file pulls in cdev.h without first pulling in
fs.h. Add a forward declaration of struct inode to cdev.h, to eliminate the
compiler warning and preserve the ability to include headers in any arbitrary
order.

We've been over this: http://lkml.org/lkml/2007/1/19/119
The patch went in into -mm and out again (why? - no reason was
attached, Andrew.)


Jan
--
ft: http://freshmeat.net/p/chaostables/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



Relevant Pages

  • [PATCH] cdev.h needs struct inode; add forward declaration
    ... compiler warning if a source file pulls in cdev.h without first pulling in ... Add a forward declaration of struct inode to cdev.h, ... compiler warning and preserve the ability to include headers in any arbitrary ...
    (Linux-Kernel)
  • Inclusion order patch
    ... This patch lets this header stand alone, ... The forward declaration of struct inode is to quiet the following ...
    (Linux-Kernel)
  • (no subject)
    ... This patch lets this header stand alone, ... The forward declaration of struct inode is to quiet the following ...
    (Linux-Kernel)
  • Re: [Long, historical] Re: Var* b or Var *b
    ... > ...an interesting proposed explanation for common styles of declaration ... contradicts the language's grammar rules ... with a commented warning. ... Another idea is a compiler warning, ...
    (alt.comp.lang.learn.c-cpp)
  • Re: Multiple DCs on single RC, and an exception in glGetIntegerv
    ... compiler warning (hah? ... the declaration ensures the DLL will actually get the pointer when I ...
    (comp.graphics.api.opengl)