• Matthew Brush's avatar
    Include what you use · 4efcbab3
    Matthew Brush yazdı
    This is a mega-commit - because most of it had to be done in one go
    otherwise some commits would fail to compile - that attempts to fix a
    few problems with Geany's includes as well as various other related
    cleanups. After this change it's easier to use includes and there's
    little worry about which order things are included in or who includes
    what.
    
    Overview of changes:
    
    * Include config.h at the start of each source file if HAVE_CONFIG_H
      is defined (and never in headers).
    * Go through each source file and make the includes section generally
      like this:
      - Always config.h first as above
      - Then if the file has a header with the same name, include that
      - Then include in alphabetical order each other internal/geany header.
      - Then include standard headers
      - Then include non-standard system headers
      - Then include GLib/GTK+ related stuff
    * Doing as above makes it easier to find implicit header include
      dependencies and it exposed quite a few weird problems with includes
      or forward declarations, fix those.
    * Make geany.h contain not much besides some defines.
      - Add a little header file "app.h" for GeanyApp and move it there
      - Move "app" global to new "app.h" file
      - Move "ignore_callback" global to "callbacks.h"
      - Move "geany_object" global to "geanyobject.h"
    * Add an include in "geany.h" for "app.h" since GeanyApp used to be
      defined there and some plugins included this header to access
      GeanyApp.
    * Include "gtkcompat.h" everywhere instead of gtk/gtk.h so that
      everywhere sees the same definitions (not a problem in practice AFAIK
      so this could be changed back if better that way.
    * Remove forward declarations from previous commits as some people
      apparently consider this bad style, despite that it reduces inter-
      header dependencies.
    
    TODO:
    * As always, to test on win32
    * As always, to test with not Autotools
    * Test plugins better, both builtin and geany-plugins, likely API/ABI bump
    * Test with various defines/flags that may change what is included
    * win32.[ch] not really touched since I couldn't test
    4efcbab3
prefix.c 6.68 KB