thliebig / openEMS

openEMS is a free and open-source electromagnetic field solver using the EC-FDTD method.
http://openEMS.de
GNU General Public License v3.0
430 stars 150 forks source link

don't segfault due to null pointer when handling dispersive materials… #40

Closed tvelliott closed 5 years ago

tvelliott commented 6 years ago

This solved the issues I was having with the LorentzMaterial type.

thliebig commented 6 years ago

I will have a look at this hopefully soon. Good job of hunting it down. One remark, you should stick to the coding style around. In the cpp source I use tabs instead of spaces as indentation. If you look at it with 4 spaces as tabs it looks really ugly now... As can already seen in the github view...

tvelliott commented 6 years ago

Thanks for the reply. Sorry about the formatting. I'll pay more attention to that next time. Just to follow up, the LorentzMaterial appears to work as it did in the past now. I only have one object of this material defined, so not sure where the null is coming from, but it appears to be working by just skipping past it.