added some comment to explain changes

git-svn-id: https://svn.kvirc.de/svn/trunk/kvirc@2873 17fca916-40b9-46aa-a4ea-0a15b648b75c

1 files changed, 10 insertions, 0 deletions

diff --git a/src/modules/perlcore/libkviperlcore.cpp b/src/modules/perlcore/libkviperlcore.cpp
index 0c66ff6f9..ad4fcc397 100644
--- a/src/modules/perlcore/libkviperlcore.cpp
+++ b/src/modules/perlcore/libkviperlcore.cpp
@@ -59,6 +59,9 @@
 // anyway, once you struggled for a couple of days with all that
 // stuff then you start getting things done... and it rox :)
 
+// Note for kvirc4: perl embedding has changed between perl 5.8 and perl 5.10.
+// This version should work nice with both, but warranty is 5.10-only.
+
 #ifdef COMPILE_PERL_SUPPORT
 	#include <EXTERN.h>
 	#include <perl.h>
@@ -85,6 +88,9 @@
 
 #include "perlcoreinterface.h"
 
+// This should be able to be rewritten in the form:
+// static PerlInterpreter *m_pInterpreter
+
 // people ... are you mad ? ... what the heck is "my_perl" ?
 #define my_perl m_pInterpreter
 
@@ -120,6 +126,10 @@ KviPerlInterpreter::~KviPerlInterpreter()
 // It SHOULD be mentioned somewhere that
 // this function is in DynaLoader.a in the perl
 // distribution and you MUST link it statically.
+
+// Update: it is no more needed as of perl 5.10, since it is
+// included in the standard libperl interface.
+
 extern "C" void boot_DynaLoader(pTHX_ CV* cv);
 
 extern "C" void xs_init(pTHX)