Great. I'll update soon and check if everything runs smoothly.
Also, looking at your changes, I'm just curious on why you removed the
implicitly_active() check. That could save a lot of
"variable.active_on_subdomain()" calls since the variable active on
all subdomains is the default case. And, if you move getting the
variable reference before the "family == SCALAR" check, you could
reuse the reference. Its trivial but just one of those things that
caught my eye. Anyway, thanks for checking in the fix.
On Thu, Jun 3, 2010 at 11:33 AM, Roy Stogner <roystgnr@...> wrote:
> On Thu, 3 Jun 2010, Vijay S. Mahadevan wrote:
>> I think this should be done for the project_vector method too.
> It should.
>> I did not do this before but if you think this is required, you can
>> add that also.
> The fix I committed to SVN is slightly simpler than your patch. It
> works on the test case you sent me, at least; let me know if it gives
> any trouble on other cases.