Dear Andriy, Sorry for breaking your use case. I wasn't aware of that. In the coming days I will progressively commit some more restructuring of core parts of GetFEM such as the model and ga_workspace classes. These changes are necessary for my implementation of internal variable condensation. Please keep an eye on what is committed to the master branch and let me know if some commit breaks any of your code in a non fixable way. Of course to avoid breakage in general we should extend our test harness to cover all use cases.
Best regards Kostas On Fri, Mar 8, 2019 at 3:11 PM Andriy Andreykiv <[email protected]> wrote: > ... additionally, > > I moved model_pb template back to the header, as it's being used in our > code > Best regards, > Andriy > > On Fri, 8 Mar 2019 at 14:22, Andriy Andreykiv <[email protected]> > wrote: > >> Dear Tetsuo Koyama, >> >> A few weeks ago you introduced [[fallthrough]] in a few places of Getfem. >> I completely agree with your intention but the problem with this feature >> is that >> it's supported only as of C++17. Many users including us are not yet >> capable to >> upgrade to the compilers that support this standard. Default compilation >> of getfem >> is currently at C++ 11, but may be GCC supports it (I don't know). >> >> May I ask you to replace [[fallthrough]] with some kind of macro that is >> only triggered >> when standard 17 is being used? >> >> Thank you, >> Andriy >> >
