added faq with an explanation of the patch-loading bug;

disabled patch-loading by default (it doesn't work anyhow);
use the pd within this directory to compile

svn path=/trunk/externals/loaders/hexloader/; revision=9836

3 files changed, 36 insertions, 3 deletions

diff --git a/FAQ.txt b/FAQ.txt
new file mode 100644
index 0000000..dbe81a3
--- /dev/null
+++ b/FAQ.txt
@@ -0,0 +1,24 @@
+============================================
+= things you might ask yourself frequently =
+============================================
+
+Q: what's this?
+A: please read the README.txt for an explanation
+
+Q: it doesn't work!
+A: try increasing the verbosity of Pd by adding the "-verbose" flag 
+   _multiple times_ to the startup flags; [hexloader] will become more
+   talkative at verbosity-level 2, which will help you (or us) to find the 
+   source of your problem
+
+Q: hexloading abstraction does not work
+A1: it is not enabled by default! you can enable it by setting the 
+    HEXLOADER_PATCHES preprocessor-define either via the Makefile or within
+    the source file.
+    the reason why it is disabled is:
+A2: hexloading abstractions (as opposed to "externals" written in other 
+    languages (anything that uses the loader-mechanism; e.g. C or lua or python
+    externals; but not Pd-abstractions)) is currently broken as it does not 
+    correctly resolve abstractions embedded within hexloaded abstractions.
+    that's a problem of Pd's abstraction-loading mechanism which i would rather
+    solve on the Pd-side
diff --git a/Makefile b/Makefile
index 3dafe2d..4f63fc7 100644
--- a/Makefile
+++ b/Makefile
@@ -3,8 +3,8 @@
 
 # path to pd
 ## change this according to your setup!
-#PDROOT=../../../pd
-PDROOT=/Net/iem/Benutzer/zmoelnig/src/pd-0.41-0test08
+PDROOT?=../../../pd
+#PDROOT=/Net/iem/Benutzer/zmoelnig/src/pd-0.41-0test08
 
 # here we find the sources of pd (and evtl. the pd.lib)
 PDSRCDIR=$(PDROOT)/src
diff --git a/hexloader.c b/hexloader.c
index 9e81f73..ef7cd1b 100644
--- a/hexloader.c
+++ b/hexloader.c
@@ -42,7 +42,15 @@
 #endif
 
 
-#define HEXLOADER_PATCHES
+/* for now hexloading abstractions does not work very well, 
+ * as it chokes when the hexloaded abstractions has nested abstractions
+ * 
+ * if you really want to enable hexloading patches, 
+ * do so via the Makefile by defining HEXLOADER_PATCHES
+ *
+ */
+//#define HEXLOADER_PATCHES
+
 
 #ifdef HEXLOADER_PATCHES
 void canvas_popabstraction(t_canvas *x);
@@ -562,6 +570,7 @@ static int hexloader_loadfile(char*pathname, char*filename, namelist_t*altnames)
   sprintf(fullfile, "%s/%s", pathname, filename);
 
   while(altname) {
+    verbose(2, "hexloader trying %s (%s)", fullfile, altname->name);
     if(hexloader_doload(fullfile, altname->name))
       return 1;