On Wed, May 30, 2012 at 3:01 PM, John Peterson <jwpeter...@gmail.com> wrote:
> On Wed, May 30, 2012 at 2:29 PM, Paul T. Bauman <ptbau...@gmail.com> wrote:
>>
>> On Wed, May 30, 2012 at 3:23 PM, John Peterson <jwpeter...@gmail.com> wrote:
>>>
>>> Here's one error:
>>>
>>> /Users/petejw/projects/libmesh_git/include/fe/fe_base.h: In
>>> constructor ‘libMesh::FEBase::FEBase(unsigned int, const
>>> libMesh::FEType&)’:
>>> /Users/petejw/projects/libmesh_git/include/fe/fe_base.h:593: error:
>>> expected identifier before ‘{’ token
>>> /Users/petejw/projects/libmesh_git/include/fe/fe_base.h:593: error:
>>> expected `(' before ‘{’ token
>>>
>>> I do not have LIBMESH_ENABLE_INFINITE_ELEMENTS defined, so this
>>> constructor list has a trailing comma?
>>
>>
>> That's what I get for trying with --enable-everything. Here's a patch for
>> fe_base.h (against trunk) that should fix that.
>
> Your second patch isn't really a great fix, since technically things
> should be initialized in the constructor in the order they are
> declared in the class, but, thinking about it for a moment, this
> constructor is just oddly-written in general:  there's not really any
> need to call all these vector default constructors explicitly.
> Probably just really old code I'm guessing...should be fine to just
> delete all of those guys.

Although svn blame tells me Roy added all these recently on 13 Jan
2011, so he will enlighten me hopefully.

-- 
John

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Libmesh-devel mailing list
Libmesh-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libmesh-devel

Reply via email to