Hello,

I wanted to communicate the creation of the branch SrcPodofoLayout[1].
The aim of the branch is creating a new source layout that allows the
removal of header wrappers while allowing to include headers directly
from podofo source directory.

In short the patch does:
- Move all source from src/* to src/podofo/*;
- Fix compilation in CMakeLists.txt;
- Fix compilation of FormTest.cpp and SignTest.cpp;
- Delete the root/podofo header wrappers folder.

It also:
- Removes unreferenced (tested with grep/strings on podofo source
tree) compilercompat header folder;
- Fixes line ending of PdfSignOutputDevice.cpp and PdfTilingPattern.h
to LF while moving files.

As with every source layout choice, I tried to identify pros and cons
of the move. Pros:
- allows removal of the header wrappers, which are not very convenient
and require maintenance;
- allows to use podofo headers just including root/src, without
polluting the include search space with other file/folders in podofo
source root directory.

Cons:
- adds another directory level in podofo source directory.

Others could be missing. I call for comments and eventually Dominik to
consider inclusion by directly merging the branch. The topic has been
discussed previously[2] in an attempt to remove the need for header
wrappers, with at least zyx manifesting a possibile interest in seeing
the layout changed.

Cheers,
Francesco

[1] https://svn.code.sf.net/p/podofo/code/podofo/branches/SrcPodofoLayout
[2] https://sourceforge.net/p/podofo/mailman/message/36236082/


_______________________________________________
Podofo-users mailing list
Podofo-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/podofo-users

Reply via email to