diff --git a/TODO b/TODO deleted file mode 100644 index de142d2889f8a5c53a0207d5d35239752ca6b2f8..0000000000000000000000000000000000000000 --- a/TODO +++ /dev/null @@ -1,100 +0,0 @@ -- allocate/free tags. -Not tested. - -OK - test blockMesh with cyclics -unitTestCases/singleCyclic/ - -OK - test cyclics sequential running. -unitTestCases/singleCyclic/ - -OK - test decomposePar -tested channel395 - -OK - FaceCellWave - unitTestCases/twoCavityCyclicForWallDistance/ - -OK - non-parallel finite volume : channelFoam - unitTestCases/channel395-splitCyclic vs. channel395-dev - -OK - parallel finite volume with processorCyclic: channelFoam - unitTestCases/channel395-splitCyclic vs. channel395-dev - -OK - preProcessing/foamUpgradeCyclics - -OK - gamg - sequential. -Tested on channel395-splitCyclic with GAMG. -OK - gamg parallel. -Tested on channel395-splitCyclic with GAMG. - -- initTransfer in GAMGprocessorInterfaces using nonblocking+tags -untested. - -OK - cyclic baffles. -Tested on t-junction-with-fan - -OK. - jumpCyclics/fanFvPatchField. All usages of jump() now need to account -for being owner() or not. -Tested on t-junction-with-fan. - -OK - regionSplit -tested on singleCyclic - -OK - pointFields on cyclics. volPointInterpolation. -tested on channel395-splitCyclic - -OK - fvMeshSubset -tested on singleCyclic - -OK - pointEdgeWave (maybe test through inversePointDistanceDiffusivity?) -tested on twoCavityCyclicForWallDistance - -OK - scotchDecomp -tested with testCalcCSR on twoCavityCyclicForWallDistance - -NOT WORKING - fvMeshDistribute to split cyclic patches into ones - with different separation. -tested on singleCyclic - -OK - test createPatch pointSync -note: only works if face-centre position of 0th faces is ok since uses -this for separation. Should in fact make cyclic planar using patch centre and -normal? -test on twoCavityCyclicForWallDistance with point (0 1 0) set to (0 1.001 0) - -NO PROBLEM - renumberMesh -It doesn't do renumbering through cyclics. - -OK - rotational cyclics. -Tested on movingCone-with-cyclics - -OK - LUscalarMatrix::convert still expects interfaces to be cyclic -tested on channel395 with 'directSolveCoarsest true;' - -OK - grep for size()/2 - -- all tutorials with cyclics: - OK - incompressible/DNS/dnsFoam/boxTurb16 - OK - incompressible/channelFoam/channel395 - slight differences due to divergence. combustion/XiFoam/les/pitzDaily3D - OK - no cyclics. combustion/fireFoam/les/smallPoolFire2D - discreteMethods/dsmcFoam/freeSpacePeriodic - discreteMethods/dsmcFoam/wedge15Ma5 - discreteMethods/molecularDynamics/mdEquilibrationFoam/periodicCubeArgon - OK - incompressible/boundaryFoam/boundaryLaunderSharma - OK - incompressible/boundaryFoam/boundaryWallFunctions - OK - incompressible/boundaryFoam/boundaryWallFunctionsProfile - OK - needs createBaffles. incompressible/pimpleFoam/t-junction-with-fan - OK - incompressible/simpleSRFFoam/mixer - OK - needs createBaffles. lagrangian/porousExplicitSourceReactingParcelFoam/filter - needs special coupledbcs. lagrangian/reactingParcelFilmFoam/multipleBoxes - - -OK - createBaffles - -- have foamUpgradeCyclics split 'value' field -- activeBaffleVelocity -- kivaToFoam/readKivaGrid.H sorts cyclics (but in incorrect order?) -- isoSurface.C -- referredCellList.C -- work out scheduled communication? -OK - add neighbourPatch checking to 16x. diff --git a/applications/solvers/multiphase/interMixingFoam/alphaEqns.H b/applications/solvers/multiphase/interMixingFoam/alphaEqns.H index 972d1d71218fba615638aedc827c701a7496e854..d302ff5147da8837458f0bdd6f2f2455a53b2995 100644 --- a/applications/solvers/multiphase/interMixingFoam/alphaEqns.H +++ b/applications/solvers/multiphase/interMixingFoam/alphaEqns.H @@ -34,7 +34,8 @@ ), mesh, dimless, - allLambda + allLambda, + false // Use slices for the couples ); diff --git a/bin/foamPackSource b/bin/foamPackSource index 4cac7b666c1a6abf4d240d8631017c10e0af11fd..e48d415b68debd5f9cd12b87b97ede790615cede 100755 --- a/bin/foamPackSource +++ b/bin/foamPackSource @@ -74,7 +74,6 @@ find -H $packDir \ -a ! -name "*.tgz" \ -a ! -name "core" \ -a ! -name "core.[1-9]*" \ - -a ! -name "log[0-9]*" \ -a ! -name "libccmio*" \ | sed \ -e "\@$packDir/lib/@d" \ diff --git a/doc/GUIDELINES b/doc/GUIDELINES deleted file mode 100644 index 5603ca3f657c13001b1e584452c3592d0be93360..0000000000000000000000000000000000000000 --- a/doc/GUIDELINES +++ /dev/null @@ -1,225 +0,0 @@ -The guidelines document the current style or a recommended style for -documenting OpenFOAM source code (.C and .H) files. - - -General -~~~~~~~ - -- For readability in the comment blocks, certain tags are used that are - translated by pre-filtering the file before sending it to doxygen. - -- The tags start in column 1, the contents follow on the next lines and - indented by 4 spaces. The filter removes the leading 4 spaces from the - following lines until the next tag that starts in column 1. - -- The 'Class' and 'Description' tags are the most important ones. - -- The first paragraph following the 'Description' will be used for the brief - description, the remaining paragraphs become the detailed description. - - -eg, - |------------------------- - | - |Class - | Foam::myClass - | - |Description - | A class for specifying the documentation style. - | - | The class is implemented as a set of recommendations that may - | sometimes be useful. - | - |------------------------- - - -- The class name must be qualified by its namespace, otherwise doxygen - will think you are documenting some other class. - -- If you don't have anything to say about the class (at the moment), - use the namespace-qualified class name for the description. - This aids with finding these under-documented classes later. - - -eg, - |------------------------- - | - |Class - | Foam::myUnderDocumentedClass - | - |Description - | Foam::myUnderDocumentedClass - | - |------------------------- - - -- Use 'Class' and 'Namespace' tags in the header files. - The Description block then applies to documenting the class. - -- Use 'InClass' and 'InNamespace' in the source files. - The Description block then applies to documenting the file itself. - -eg, - |------------------------- - | - |InClass - | Foam::myClass - | - |Description - | Implements the read and writing of files. - | - |------------------------- - - -Doxygen Special Commands -~~~~~~~~~~~~~~~~~~~~~~~~ - -Doxygen has a large number of special commands with a '\' prefix or -a (alternatively) an '@' prefix. - -The '@' prefix form is recommended for most doxygen specials, since it has -the advantage of standing out. It also happens to be what projects like gcc -and VTK are using. - -The '\' prefix form, however, looks a bit better for the '\n' newline command -and when escaping single characters - eg, '\@', '\<', '\>', etc. - - -Since the filtering removes the leading 4 spaces within the blocks, -the doxygen commmands can be inserted within the block without problems. - -eg, - |------------------------- - | - |InClass - | Foam::myClass - | - |Description - | Implements the read and writing of files. - | - | An example input file: - | @verbatim - | patchName - | { - | type myPatchType; - | refValue 100; - | value uniform 1; - | } - | @endverbatim - | - | Within the implementation, a loop over all patches is done: - | @code - | forAll(patches, patchI) - | { - | ... // some operation - | } - | @endcode - | - |------------------------- - - -HTML Special Commands -~~~~~~~~~~~~~~~~~~~~~ - -Since Doxygen also handles HTML tags to a certain extent, the angle brackets -need quoting in the documentation blocks. Non-HTML tags cause doxygen to -complain, but seem to work anyhow. - -eg, - The template with type <HR> is a bad example. - - The template with type \<HR\> is a better example. - - The template with type <Type> causes doxygen to complain about - an unknown html type, but it seems to work okay anyhow. - - - -Documenting Namespaces -~~~~~~~~~~~~~~~~~~~~~~ - -- If namespaces are explictly declared with the Namespace() macro, - they should be documented there. - -- If the namespaces is used to hold sub-models, the namespace can be - documented in the same file as the class with the model selector. - eg, - documented namespace 'Foam::functionEntries' within the - class 'Foam::functionEntry' - -- If nothing else helps, find some sensible header. - eg, - namespace 'Foam' is documented in the foamVersion.H file - - -Documenting Typedefs and classes defined via macros -~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ - -... not yet properly resolved - - - -Documenting Applications -~~~~~~~~~~~~~~~~~~~~~~~~ - -Any number of classes might be defined by a particular application, but -these classes will not, however, be available to other parts of OpenFOAM. At -the moment, the sole purpuse for running doxygen on the applications is to -extract program usage information for the '-doc' option. - -The documentation for a particular application is normally contained within -the first comment block in a .C source file. The solution is this to invoke -a special filter for the "applications/{solver,utilities}" directories that -only allows the initial comment block for the .C files through. - -The layout of the application documentation has not yet been finalized, but -foamToVTK shows an initial attempt. - - -Ignored files and directories -~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ - -Ignore directories of linked files - - */lnInclude/* - -Ignore test directories - - */t/* - -Ignore applications that clutter everything - -Ignore application-specific classes - - */applications/utilities/*.H - - */applications/solvers/*.H - - -Orthography (always good for a flame-war) -~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ - -Given the origins of OpenFOAM, the British spellings (eg, neighbour and not -neighbor) are generally favoured. For code sections that interact with -external libraries, it can be useful to adopt American spellings, especially -for names that constitute a significant part of the external library - eg, -'color' within graphics sub-systems. - -Both '-ize' and the '-ise' variant are found in the code comments. -If used as a variable or class method name, it is probably better to use '-ize', -which is considered the main form by the Oxford University Press. - Eg, - myClass.initialize() - - -The word "its" (possesive) vs. "it's" (colloquial for "it is" or "it has") -seems to confuse non-native (and some native) English speakers. -It is better to donate the extra keystrokes and write "it is" or "it has". -Any remaining "it's" are likely an incorrect spelling of "its". - - - -Housekeeping -~~~~~~~~~~~~ - -The doc/Doxygen/tools directory contains miscellaneous scripts for finding -and possibly repairing documentation issues. - - -Updated: 2009-11-27 diff --git a/CHANGES-splitCyclic b/doc/changes/splitCyclic.txt similarity index 100% rename from CHANGES-splitCyclic rename to doc/changes/splitCyclic.txt diff --git a/doc/codingStyleGuide.org b/doc/codingStyleGuide.org index abd7bd1a0095736f204b12b7a484a80aca165d20..94a1b51536e368c407fa98b764a6afab38931de8 100644 --- a/doc/codingStyleGuide.org +++ b/doc/codingStyleGuide.org @@ -2,7 +2,7 @@ # #+TITLE: OpenFOAM C++ style guide #+AUTHOR: OpenCFD Ltd. -#+DATE: April 2010 +#+DATE: May 2010 #+LINK: http://www.opencfd.co.uk #+OPTIONS: author:nil ^:{} @@ -13,9 +13,9 @@ + The normal indentation is 4 spaces per logical level. + Use spaces for indentation, not tab characters. + Avoid trailing whitespace. - + The body of control statements (eg, if, else, while, etc). + + The body of control statements (eg, =if=, =else=, =while=, etc). always delineated with brace brackets. A possible exception can be - made with 'break' or 'continue' as part of a control structure. + made with =break= or =continue= as part of a control structure. + stream output + =<<= is always four characters after the start of the stream, @@ -72,16 +72,16 @@ \*---------------------------------------------------------------------------*/ #+END_EXAMPLE -*** The =.H= Files +*** The /.H/ Files + header file spacing + Leave two empty lines between sections - (as per functions in the =.C= file etc) + (as per functions in the /.C/ file etc) - + use "//- Comment" comments in header file + + use =//- Comment= comments in header file + add descriptions to class data and functions + destructor + If adding a comment to the destructor - - use //- and code as a normal function: + use =//-= and code as a normal function: #+BEGIN_EXAMPLE //- Destructor @@ -89,11 +89,11 @@ #+END_EXAMPLE + inline functions - + Use inline functions where appropriate in a separate classNameI.H file. + + Use inline functions where appropriate in a separate /classNameI.H/ file. Avoid cluttering the header file with function bodies. -*** The =.C= Files - + Do not open/close namespaces in a =.C= file +*** The /.C/ Files + + Do not open/close namespaces in a /.C/ file + Fully scope the function name, i.e. #+BEGIN_EXAMPLE @@ -113,7 +113,7 @@ EXCEPTION - When there are multiple levels of namespace, they may be used in the =.C= + When there are multiple levels of namespace, they may be used in the /.C/ file, i.e. #+BEGIN_EXAMPLE @@ -138,7 +138,7 @@ + const Use everywhere it is applicable. - + variable initialisation using "=" + + variable initialisation using = : const className& variableName = otherClass.data(); @@ -169,7 +169,7 @@ } #+END_EXAMPLE - NOT (no space between "if" and "(") + NOT (no space between =if= and =(=) #+BEGIN_EXAMPLE if(condition) @@ -201,7 +201,7 @@ } #+END_EXAMPLE - NOT (no space between "for" and "(") + NOT (no space between =for= and =(=) #+BEGIN_EXAMPLE for(i = 0; i < maxI; i++) @@ -245,7 +245,7 @@ **** Splitting return type and function name + split initially after the function return type and left align - + do not put "const" onto its own line - use a split to keep it with + + do not put =const= onto its own line - use a split to keep it with the function name and arguments. so @@ -277,7 +277,7 @@ #+END_EXAMPLE + if it needs to be split again, split at the function name (leaving - behind the preceding scoping "::"s), and again, left align, i.e. + behind the preceding scoping =::=s), and again, left align, i.e. For example, @@ -326,16 +326,19 @@ *** Maths and Logic + operator spacing - + a + b, a - b - + a*b, a/b - + a & b, a ^ b - + a = b, a != b - + a < b, a > b, a >= b, a <= b - + a || b, a && b + +#+BEGIN_EXAMPLE + a + b, a - b + a*b, a/b + a & b, a ^ b + a = b, a != b + a < b, a > b, a >= b, a <= b + a || b, a && b +#+END_EXAMPLE + splitting formulae over several lines - Split and indent as per "splitting long lines at an "="" + Split and indent as per "splitting long lines at an =" with the operator on the lower line. Align operator so that first variable, function or bracket on the next line is 4 spaces indented i.e. @@ -525,9 +528,9 @@ option. The documentation for a particular application is normally contained - within the first comment block in a =.C= source file. The solution is this - to invoke a special filter for the "applications/{solver,utilities}" - directories that only allows the initial comment block for the =.C= files + within the first comment block in a /.C/ source file. The solution is this + to invoke a special filter for the "/applications/{solver,utilities}/" + directories that only allows the initial comment block for the /.C/ files through. The layout of the application documentation has not yet been finalized, @@ -551,10 +554,7 @@ myClass.initialize() #+END_EXAMPLE - The word "its" (possesive) vs. "it's" (colloquial for "it is" or "it has") seems to confuse non-native (and some native) English speakers. It is better to donate the extra keystrokes and write "it is" or "it has". Any remaining "it's" are likely an incorrect spelling of "its". - - diff --git a/doc/codingStyleGuide.pdf b/doc/codingStyleGuide.pdf new file mode 100644 index 0000000000000000000000000000000000000000..e7119e7fdcbeef1263bfaeabe05adfc15edf5ce7 Binary files /dev/null and b/doc/codingStyleGuide.pdf differ diff --git a/unitTestCases/README.txt b/unitTestCases/README.txt deleted file mode 100644 index 1f2ce33a509c19453c67b00029ad5b2a04146787..0000000000000000000000000000000000000000 --- a/unitTestCases/README.txt +++ /dev/null @@ -1,18 +0,0 @@ -channel395/ - - splitCyclic version - dev/ - - dev version - -singleCyclic - - single cyclic z=0 to z=4 - dev/ - - dev version - -t-junction-with-fan - - t-junction tutorial to test baffles and fan-cyclics - dev/ - - dev version - -twoCavityCyclicForWallDistance - - single cyclic. bottom split into two patches to see how - wallDistance transfers through cyclic.