May I please ask for your opinion on the following code:
May I please ask for your opinion on the following code:
https://gist.github.com/jkour/b45d3b35d9618cc4fac78452d2d3c272
I am trying to find a generalised way to create global classes in my app and (given the my recent exploration into generics) I came up with the following code.
There are two points unclear to me:
1. Is the use of Critical Section correct? Would that make the code thread-safe?
2. I get a memory leak but I can't figure out which object or something is not freed
https://gist.github.com/jkour/b45d3b35d9618cc4fac78452d2d3c272
https://gist.github.com/jkour/b45d3b35d9618cc4fac78452d2d3c272
I am trying to find a generalised way to create global classes in my app and (given the my recent exploration into generics) I came up with the following code.
There are two points unclear to me:
1. Is the use of Critical Section correct? Would that make the code thread-safe?
2. I get a memory leak but I can't figure out which object or something is not freed
https://gist.github.com/jkour/b45d3b35d9618cc4fac78452d2d3c272
I get the leak on 10.1 Update 2 as well. Strange you can't see it though
ReplyDeleteAgustin Ortu wow, they're finally plugging holes that have been there for a very long time!
ReplyDeleteThe leak does occur on Berlin. But the report just works fine on D2010 without any extra work
ReplyDelete