On Thu, 16 Sep 2010, Roy Stogner wrote:

On Thu, 16 Sep 2010, Tim Kroeger wrote:

This keep annoying me continuously, so I tracked it back and found that the attached patch solves the problem, although I don't know why. I'll check that in if nobody objects.

I wonder if doxygen is getting confused by my spartan ifdefs there?
Unless you take one preprocessor path or the other the braces don't
even match up; of course the compiler will be fine but perhaps
doxygen's parser gets confused by it.  Maybe try making a copy of that
"return false;\n}" and put the two copies inside the two halfs of the
ifdef/else?  That would be a cleaner solution.

That does not solve the problem. However, the attached method does and is certainly cleaner than my first solution. I'll check that in if you agree (or you can check it in yourself if you like).

Will the documentation be regenerated and uploaded automatically then?

No; "make doc upload" does that.  You should have write permissions to
the svn web space, but if it gives you any trouble let us know.

The problem is that this will also upload the documentation for my subset_solve work. Hence I suggest that you upload it this time (provided that your working copy does not have any local changes). I will know what to do later when the subset_solve stuff is finished.

Best Regards,

Tim

--
Dr. Tim Kroeger
CeVis -- Center of Complex Systems and Visualization
University of Bremen              [email protected]
Universitaetsallee 29             [email protected]
D-28359 Bremen                             Phone +49-421-218-7710
Germany                                    Fax   +49-421-218-4236
Index: include/geom/elem.h
===================================================================
--- include/geom/elem.h (Revision 3957)
+++ include/geom/elem.h (Arbeitskopie)
@@ -1383,10 +1383,14 @@
 
 
 
+inline
+bool Elem::is_ancestor_of(const Elem *
 #ifdef LIBMESH_ENABLE_AMR
-inline
-bool Elem::is_ancestor_of(const Elem *descendant) const
+                         descendant
+#endif
+                         ) const
 {
+#ifdef LIBMESH_ENABLE_AMR
   const Elem *e = descendant;
   while (e)
     {
@@ -1394,10 +1398,6 @@
         return true;
       e = e->parent();
     }
-#else
-inline
-bool Elem::is_ancestor_of(const Elem *) const
-{
 #endif
   return false;
 }
------------------------------------------------------------------------------
Start uncovering the many advantages of virtual appliances
and start using them to simplify application deployment and
accelerate your shift to cloud computing.
http://p.sf.net/sfu/novell-sfdev2dev
_______________________________________________
Libmesh-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/libmesh-devel

Reply via email to