From: David K. <dkn...@se...> - 2013-06-17 06:34:10
|
Random question: why does FEMContext::pre_fe_reinit take a System& argument when DiffContext already stores a reference to the system? |
From: Paul T. B. <ptb...@gm...> - 2013-06-20 19:16:08
|
Roy's going to have to chime in here on the decision, but it looks like the cached System reference is private in DiffContext, so FEMContext can't access it. I'd be in favor of moving it to protected. On Mon, Jun 17, 2013 at 1:33 AM, David Knezevic <dkn...@se...>wrote: > Random question: why does FEMContext::pre_fe_reinit take a System& > argument when DiffContext already stores a reference to the system? > > > > ------------------------------------------------------------------------------ > This SF.net email is sponsored by Windows: > > Build for Windows Store. > > http://p.sf.net/sfu/windows-dev2dev > _______________________________________________ > Libmesh-devel mailing list > Lib...@li... > https://lists.sourceforge.net/lists/listinfo/libmesh-devel > |
From: David K. <dkn...@se...> - 2013-06-20 22:51:36
|
On 06/21/2013 03:16 AM, Paul T. Bauman wrote: > Roy's going to have to chime in here on the decision, but it looks > like the cached System reference is private in DiffContext, so > FEMContext can't access it. I'd be in favor of moving it to protected. There's a const accessor in DiffContext: const System& get_system() const > > > On Mon, Jun 17, 2013 at 1:33 AM, David Knezevic > <dkn...@se... <mailto:dkn...@se...>> wrote: > > Random question: why does FEMContext::pre_fe_reinit take a System& > argument when DiffContext already stores a reference to the system? > > > ------------------------------------------------------------------------------ > This SF.net email is sponsored by Windows: > > Build for Windows Store. > > http://p.sf.net/sfu/windows-dev2dev > _______________________________________________ > Libmesh-devel mailing list > Lib...@li... > <mailto:Lib...@li...> > https://lists.sourceforge.net/lists/listinfo/libmesh-devel > > |
From: Paul T. B. <ptb...@gm...> - 2013-06-20 23:27:00
|
Derp. Only other thing that occurs to me is that it could be related to the ALE stuff Roy had started on. That way same method can deal with both the "primary" system and the mesh system. On Jun 20, 2013, at 5:51 PM, David Knezevic <dkn...@se...> wrote: > On 06/21/2013 03:16 AM, Paul T. Bauman wrote: >> Roy's going to have to chime in here on the decision, but it looks like the cached System reference is private in DiffContext, so FEMContext can't access it. I'd be in favor of moving it to protected. > > There's a const accessor in DiffContext: > > const System& get_system() const > > > > > > >> >> >> On Mon, Jun 17, 2013 at 1:33 AM, David Knezevic <dkn...@se...> wrote: >> Random question: why does FEMContext::pre_fe_reinit take a System& >> argument when DiffContext already stores a reference to the system? >> >> >> ------------------------------------------------------------------------------ >> This SF.net email is sponsored by Windows: >> >> Build for Windows Store. >> >> http://p.sf.net/sfu/windows-dev2dev >> _______________________________________________ >> Libmesh-devel mailing list >> Lib...@li... >> https://lists.sourceforge.net/lists/listinfo/libmesh-devel >> > |
From: David K. <dkn...@se...> - 2013-06-20 23:33:18
|
On 06/21/2013 07:26 AM, Paul T. Bauman wrote: > Only other thing that occurs to me is that it could be related to the > ALE stuff Roy had started on. That way same method can deal with both > the "primary" system and the mesh system. OK, that makes sense. I always pass the cached system pre_fe_reinit, so I just wanted to make sure I wasn't missing something. |