Restructure the header files

Think CEF could benefit from a new feature or capability? Discuss CEF feature requests here.

Restructure the header files

Postby simast » Tue Oct 01, 2013 1:33 am

CEF might have the weirdest header file structure I have ever seen in a C/C++ library. What's up with the hardcoded "include" directory? E.g. all header files have:

#include "include/cef_base.h"

Which means you have to place CEF header files in an awkward directory structure, such as:

lib/include/include <--- CEF
lib/include/boost
lib/include/something
etc

A change to header files right now would mean a BC break (but frankly quite a minor one if all you have to do is update your #include directives).
simast
Techie
 
Posts: 15
Joined: Mon Sep 30, 2013 3:35 pm

Re: Restructure the header files

Postby Czarek » Tue Oct 01, 2013 2:20 am

Put it in lib/include/cef/ => lib/include/cef/include/*.h - what's the problem?

All the header files are prefixed with "cef_" so the "include" directory name is not really an issue, it won't
break anything.

Making changes to the files structure not a good idea, just think of all the projects that already use CEF.
Maintainer of the CEF Python, PHP Desktop and CEF C API projects. My LinkedIn.
User avatar
Czarek
Virtuoso
 
Posts: 1927
Joined: Sun Nov 06, 2011 2:12 am


Return to Feature Request Forum

Who is online

Users browsing this forum: No registered users and 17 guests