LSUThorns/Vectors: Update Kranc-specific code

Comments (11)

Erik Schnetter reporter
- changed status to open
- removed comment
- 2011-11-26T10:34:52+00:00
Ian Hinder
- removed comment
I don't quite understand the undefinition of E. If the symbol is used elsewhere with a different meaning, how does that interact with Kranc? Could you elaborate?
- 2012-01-16T09:42:58+00:00
Erik Schnetter reporter
- removed comment
1. define does not have name scopes. If E is defined, we assume it comes from Kranc, and redefine it to its proper value with vectorisation (i.e. with ToReal). If it is not defined, then we don't define it here either.
If we #define E here unconditionally, and some code tries to use it as a variable/type name, then this will fail. This would insert the identifier E into all source code, even those not using Kranc.
- 2012-01-16T09:56:09+00:00
Ian Hinder
- removed comment
OK I see. Would it make sense to change the conditional to #ifdef KRANC_C so that you can be sure you are only affecting Kranc thorns? It might be that other header files also define E for their own purposes. For Kranc-generated thorns, we can be fairly sure that this is not the case.

Now that I think about it some more, why is this replacement handled here by macros? Wouldn't it make more sense to have E -> ToReal[E] in Kranc when vectorisation is enabled (and similarly for Pi)? Then Vectors would not need to know about a Kranc-specific symbol, and Kranc would be treating these constants just like it treats parameters (for example).
- 2012-01-17T03:27:16+00:00
Erik Schnetter reporter
- removed comment
Yes, using KRANC_C would be better.

Yes, doing this in Kranc would also work.
- 2012-01-17T09:51:18+00:00
Frank Löffler
- changed status to open
- removed comment
- 2012-01-27T09:02:01+00:00
Barry Wardell
- removed comment
So the suggestion is to have something like to two patches I have just attached (one for Vectors, and the other for Kranc)?
- 2012-02-04T17:50:20+00:00
Erik Schnetter reporter
- removed comment
Yes, similar to these patches.

I plan to commit my new version of the patch (without your patches) later today, and you could apply them then.
- 2012-02-04T18:16:51+00:00
Erik Schnetter reporter
- removed comment
Applied.
- 2012-02-04T18:21:26+00:00
Barry Wardell
- changed status to resolved
- removed comment
- 2012-04-26T12:24:11+00:00
Roland Haas
- edited description
- changed status to closed
- 2019-02-21T19:56:05+00:00
Log in to comment