[pHash-support] Java usage of pHash.java
Sebastian Gallese
sebastiangallese at gmail.com
Fri Jan 15 01:03:06 PST 2010
Error from removing static from ph_free_datapoint. Meh, no worries.
Thanks,
Seb
ni.lo -MD -MP -MF .deps/pHash-jni.Tpo -c pHash-jni.cpp -fPIC -DPIC -o
.libs/pHash-jni.o
pHash-jni.cpp: In function ‘jboolean
Java_pHash_00024MVPTree_create(JNIEnv*, _jobject*, _jobjectArray*)’:
pHash-jni.cpp:155: error: ‘ph_free_datapoint’ was not declared in this scope
pHash-jni.cpp:174: error: ‘ph_free_datapoint’ was not declared in this scope
pHash-jni.cpp:204: error: ‘ph_free_datapoint’ was not declared in this scope
pHash-jni.cpp:220: error: ‘ph_free_datapoint’ was not declared in this scope
pHash-jni.cpp: In function ‘_jobjectArray*
Java_pHash_00024MVPTree_query(JNIEnv*, _jobject*, _jobject*, jfloat,
jint)’:
pHash-jni.cpp:324: error: ‘ph_free_datapoint’ was not declared in this scope
pHash-jni.cpp: In function ‘jboolean
Java_pHash_00024MVPTree_add(JNIEnv*, _jobject*, _jobjectArray*)’:
pHash-jni.cpp:420: error: ‘ph_free_datapoint’ was not declared in this scope
make[3]: *** [pHash-jni.lo] Error 1
make[3]: Leaving directory `/home/kdge/pHash-0.7.2/bindings/java'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/home/kdge/pHash-0.7.2/bindings'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/home/kdge/pHash-0.7.2'
make: *** [all] Error 2
On Wed, Jan 13, 2010 at 4:21 PM, Evan Klinger <eklinger at phash.org> wrote:
> One more quick test case:
> remove static from ph_free_datapoint in pHash.h.
> Otherwise I will look into it this weekend.
>
> On Wed, Jan 13, 2010 at 12:49 PM, Sebastian Gallese
> <sebastiangallese at gmail.com> wrote:
>> Evan,
>>
>> Heh, there is no such thing as "once and for all." Please do not feel
>> obliged to hack away until you have some legitimate free time.
>>
>> Thank you so much for the help!
>>
>> -Seb
>>
>> libtool: link: (cd ".libs" && rm -f "libpHash.so.0" && ln -s
>> "libpHash.so.0.0.0" "libpHash.so.0")
>> libtool: link: (cd ".libs" && rm -f "libpHash.so" && ln -s
>> "libpHash.so.0.0.0" "libpHash.so")
>> libtool: link: ar cru .libs/libpHash.a pHash.o audiophash.o ph_fft.o
>> cimgffmpeg.o
>> libtool: link: ranlib .libs/libpHash.a
>> libtool: link: ( cd ".libs" && rm -f "libpHash.la" && ln -s
>> "../libpHash.la" "libpHash.la" )
>> make[2]: Leaving directory `/home/kdge/pHash-0.7.2/src'
>> Making all in bindings
>> make[2]: Entering directory `/home/kdge/pHash-0.7.2/bindings'
>> Making all in java
>> make[3]: Entering directory `/home/kdge/pHash-0.7.2/bindings/java'
>> /bin/bash ../../libtool --tag=CXX --mode=compile g++ -DHAVE_CONFIG_H
>> -I. -I../.. -I../../src -I/usr/lib/jvm/java-6-sun/include
>> -I/usr/local/include -DHAVE_IMAGE_HASH=1 -DHAVE_VIDEO_HASH=1
>> -DHAVE_AUDIO_HASH=1 -Dcimg_use_jpeg -Dcimg_use_png -O2 -ffast-math
>> -O3 -MT pHash-jni.lo -MD -MP -MF .deps/pHash-jni.Tpo -c -o
>> pHash-jni.lo pHash-jni.cpp
>> libtool: compile: g++ -DHAVE_CONFIG_H -I. -I../.. -I../../src
>> -I/usr/lib/jvm/java-6-sun/include -I/usr/local/include
>> -DHAVE_IMAGE_HASH=1 -DHAVE_VIDEO_HASH=1 -DHAVE_AUDIO_HASH=1
>> -Dcimg_use_jpeg -Dcimg_use_png -O2 -ffast-math -O3 -MT pHash-jni.lo
>> -MD -MP -MF .deps/pHash-jni.Tpo -c pHash-jni.cpp -fPIC -DPIC -o
>> .libs/pHash-jni.o
>> pHash-jni.cpp: In function ‘jboolean
>> Java_pHash_00024MVPTree_create(JNIEnv*, _jobject*, _jobjectArray*)’:
>> pHash-jni.cpp:155: error: ‘ph_free_datapoint’ was not declared in this scope
>> pHash-jni.cpp:174: error: ‘ph_free_datapoint’ was not declared in this scope
>> pHash-jni.cpp:204: error: ‘ph_free_datapoint’ was not declared in this scope
>> pHash-jni.cpp:220: error: ‘ph_free_datapoint’ was not declared in this scope
>> pHash-jni.cpp: In function ‘_jobjectArray*
>> Java_pHash_00024MVPTree_query(JNIEnv*, _jobject*, _jobject*, jfloat,
>> jint)’:
>> pHash-jni.cpp:324: error: ‘ph_free_datapoint’ was not declared in this scope
>> pHash-jni.cpp: In function ‘jboolean
>> Java_pHash_00024MVPTree_add(JNIEnv*, _jobject*, _jobjectArray*)’:
>> pHash-jni.cpp:420: error: ‘ph_free_datapoint’ was not declared in this scope
>> make[3]: *** [pHash-jni.lo] Error 1
>> make[3]: Leaving directory `/home/kdge/pHash-0.7.2/bindings/java'
>> make[2]: *** [all-recursive] Error 1
>> make[2]: Leaving directory `/home/kdge/pHash-0.7.2/bindings'
>> make[1]: *** [all-recursive] Error 1
>> make[1]: Leaving directory `/home/kdge/pHash-0.7.2'
>> make: *** [all] Error 2
>>
>> On Wed, Jan 13, 2010 at 3:39 PM, Evan Klinger <eklinger at phash.org> wrote:
>>> Looks like there are 2 more that need to be changed that aren't
>>> static, ph_compare_images and ph_image_digest. The ones that take a
>>> CImg parameter, prepend them with _
>>> I think that might solve this once and for all.
>>>
>>> On Wed, Jan 13, 2010 at 12:24 PM, Sebastian Gallese
>>> <sebastiangallese at gmail.com> wrote:
>>>> I searched for "static" and placed an underscore in front of each function.
>>>>
>>>> Looks like overloading is the issue with the error below. Anything I should do?
>>>>
>>>> Attached is my pHash.h file.
>>>>
>>>> In file included from pHash.cpp:25:
>>>> pHash.h:263: error: declaration of C function ‘int
>>>> ph_image_digest(const char*, double, double, Digest&, int)’ conflicts
>>>> with
>>>> pHash.h:253: error: previous declaration ‘int ph_image_digest(const
>>>> cimg_library::CImg<unsigned char>&, double, double, Digest&, int)’
>>>> here
>>>> In file included from pHash.cpp:25:
>>>> pHash.h:288: error: declaration of C function ‘int
>>>> ph_compare_images(const char*, const char*, double&, double, double,
>>>> int, double)’ conflicts with
>>>> pHash.h:276: error: previous declaration ‘int ph_compare_images(const
>>>> cimg_library::CImg<unsigned char>&, const cimg_library::CImg<unsigned
>>>> char>&, double&, double, double, int, double)’ here
>>>> make[2]: *** [pHash.lo] Error 1
>>>> make[2]: Leaving directory `/home/kdge/pHash-0.7.2/src'
>>>> make[1]: *** [all-recursive] Error 1
>>>> make[1]: Leaving directory `/home/kdge/pHash-0.7.2'
>>>> make: *** [all] Error 2
>>>>
>>>> Thanks,
>>>> Seb
>>>>
>>>>
>>>>
>>>> On Tue, Jan 12, 2010 at 6:31 PM, Evan Klinger <eklinger at phash.org> wrote:
>>>>> Yeah it looks like I didn't rename them all. For the static functions,
>>>>> try prepending an _ to the function name to get rid of these errors.
>>>>> This weekend I will have more time to resolve all these issues so you
>>>>> can move forward rather than debugging.
>>>>>
>>>>> On Tue, Jan 12, 2010 at 2:10 PM, Sebastian Gallese
>>>>> <sebastiangallese at gmail.com> wrote:
>>>>>> Evan,
>>>>>>
>>>>>> Thanks for sending me the updated pHash.h file. I replaced my old
>>>>>> pHash.h file and I got a new error message on building the project:
>>>>>>
>>>>>> make all-recursive
>>>>>> make[1]: Entering directory `/home/kdge/pHash-0.7.2'
>>>>>> Making all in src
>>>>>> make[2]: Entering directory `/home/kdge/pHash-0.7.2/src'
>>>>>> /bin/bash ../libtool --tag=CXX --mode=compile g++ -DHAVE_CONFIG_H -I.
>>>>>> -I.. -I/usr/lib/jvm/java-6-sun/include -I/usr/local/include
>>>>>> -DHAVE_IMAGE_HASH=1 -DHAVE_VIDEO_HASH=1 -DHAVE_AUDIO_HASH=1
>>>>>> -Dcimg_use_jpeg -Dcimg_use_png -O2 -ffast-math -O3 -MT pHash.lo -MD
>>>>>> -MP -MF .deps/pHash.Tpo -c -o pHash.lo pHash.cpp
>>>>>> libtool: compile: g++ -DHAVE_CONFIG_H -I. -I..
>>>>>> -I/usr/lib/jvm/java-6-sun/include -I/usr/local/include
>>>>>> -DHAVE_IMAGE_HASH=1 -DHAVE_VIDEO_HASH=1 -DHAVE_AUDIO_HASH=1
>>>>>> -Dcimg_use_jpeg -Dcimg_use_png -O2 -ffast-math -O3 -MT pHash.lo -MD
>>>>>> -MP -MF .deps/pHash.Tpo -c pHash.cpp -fPIC -DPIC -o .libs/pHash.o
>>>>>> In file included from pHash.cpp:25:
>>>>>> pHash.h:263: error: declaration of C function ‘int
>>>>>> ph_image_digest(const char*, double, double, Digest&, int)’ conflicts
>>>>>> with
>>>>>> pHash.h:253: error: previous declaration ‘int ph_image_digest(const
>>>>>> cimg_library::CImg<unsigned char>&, double, double, Digest&, int)’
>>>>>> here
>>>>>> In file included from pHash.cpp:25:
>>>>>> pHash.h:288: error: declaration of C function ‘int
>>>>>> ph_compare_images(const char*, const char*, double&, double, double,
>>>>>> int, double)’ conflicts with
>>>>>> pHash.h:276: error: previous declaration ‘int ph_compare_images(const
>>>>>> cimg_library::CImg<unsigned char>&, const cimg_library::CImg<unsigned
>>>>>> char>&, double&, double, double, int, double)’ here
>>>>>> pHash.h:398: error: declaration of C function ‘MVPRetCode
>>>>>> ph_query_mvptree(MVPFile*, DP*, int, float, DP**, int*)’ conflicts
>>>>>> with
>>>>>> pHash.h:386: error: previous declaration ‘MVPRetCode
>>>>>> ph_query_mvptree(MVPFile*, DP*, int, float, DP**, int*, int)’ here
>>>>>> pHash.h:417: error: declaration of C function ‘MVPRetCode
>>>>>> ph_save_mvptree(MVPFile*, DP**, int)’ conflicts with
>>>>>> pHash.h:408: error: previous declaration ‘FileIndex*
>>>>>> ph_save_mvptree(MVPFile*, DP**, int, int, int)’ here
>>>>>> pHash.h:433: error: declaration of C function ‘int
>>>>>> ph_add_mvptree(MVPFile*, DP**, int)’ conflicts with
>>>>>> pHash.h:425: error: previous declaration ‘MVPRetCode
>>>>>> ph_add_mvptree(MVPFile*, DP*, int)’ here
>>>>>> make[2]: *** [pHash.lo] Error 1
>>>>>> make[2]: Leaving directory `/home/kdge/pHash-0.7.2/src'
>>>>>> make[1]: *** [all-recursive] Error 1
>>>>>> make[1]: Leaving directory `/home/kdge/pHash-0.7.2'
>>>>>> make: *** [all] Error 2
>>>>>>
>>>>>> Full output at http://gist.github.com/275677
>>>>>>
>>>>>> Don't know much about C, but it looks like it might have to do with
>>>>>> the "same name but different parameter" methods you mentioned before.
>>>>>>
>>>>>> Let me know if there is anything else you want me to test out.
>>>>>>
>>>>>> Thanks,
>>>>>>
>>>>>> Seb
>>>>>>
>>>>>> On Tue, Jan 12, 2010 at 1:27 PM, Evan Klinger <eklinger at phash.org> wrote:
>>>>>>> Sebastian,
>>>>>>> I don't think you can put it at the very beginning. It should be after
>>>>>>> the #includes. You will also run into another error because some of
>>>>>>> the functions have the same name but different parameters, but extern
>>>>>>> "C" only expects unique function names. Please try using this header
>>>>>>> instead and see if it works. Hang in there - we will get it working
>>>>>>> for you.
>>>>>>>
>>>>>>> Evan
>>>>>>>
>>>>>>> On Tue, Jan 12, 2010 at 10:20 AM, Sebastian Gallese
>>>>>>> <sebastiangallese at gmail.com> wrote:
>>>>>>>> Hi Evan,
>>>>>>>>
>>>>>>>> In pHash-0.7.2/src/pHash.h I added
>>>>>>>> extern "C" {
>>>>>>>> to the very top of the file and
>>>>>>>> }
>>>>>>>> to the very bottom of the file. I got the idea to do that from reading
>>>>>>>> http://stackoverflow.com/questions/1041866/extern-c
>>>>>>>> and
>>>>>>>> http://www.parashift.com/c++-faq-lite/mixing-c-and-cpp.html
>>>>>>>>
>>>>>>>> I then ran
>>>>>>>> sudo CPPFLAGS=-I/usr/lib/jvm/java-6-sun/include ./configure
>>>>>>>> --enable-java && sudo make && sudo make install
>>>>>>>>
>>>>>>>> and I got the output that and uploaded it to http://gist.github.com/275427
>>>>>>>>
>>>>>>>> It goes haywire around here:
>>>>>>>> In file included from /usr/include/c++/4.4/cmath:44,
>>>>>>>> from /usr/include/CImg.h:75,
>>>>>>>> from pHash.h:47,
>>>>>>>> from pHash.cpp:25:
>>>>>>>> /usr/include/c++/4.4/bits/cpp_type_traits.h:71: error: template with C linkage
>>>>>>>> /usr/include/c++/4.4/bits/cpp_type_traits.h:81: error: template with C linkage
>>>>>>>>
>>>>>>>> Any thoughts or suggestions would be great!
>>>>>>>>
>>>>>>>> Thank you,
>>>>>>>> Seb
>>>>>>>>
>>>>>>>> On Mon, Jan 11, 2010 at 2:00 PM, Evan Klinger <eklinger at phash.org> wrote:
>>>>>>>>> Sebastian,
>>>>>>>>> It looks like the entire API might need to be wrapped in extern "C" to
>>>>>>>>> get this to work. To be honest, you're the first that is attempting to
>>>>>>>>> use the Java bindings so it's a learning process for both of us. There
>>>>>>>>> is an example on how to use the functions in pHash.java in main(). Try
>>>>>>>>> adding the extern "C" to all the functions in pHash.h and rebuild and
>>>>>>>>> see if that helps. Sorry for the trouble.
>>>>>>>>> Thanks
>>>>>>>>>
>>>>>>>>> On Mon, Jan 11, 2010 at 9:30 AM, Sebastian Gallese
>>>>>>>>> <sebastiangallese at gmail.com> wrote:
>>>>>>>>>> Hi Evan,
>>>>>>>>>>
>>>>>>>>>> I'm unsure of how I should use pHash with Java. Do you have some
>>>>>>>>>> basic test code or demo code to show me the basic usage?
>>>>>>>>>>
>>>>>>>>>> For example, right now I'm trying to call on pHash to get a hash from an image.
>>>>>>>>>>
>>>>>>>>>> I threw the file from pHash-0.7.2/bindings/java/pHash.java into a
>>>>>>>>>> package in Eclipse.
>>>>>>>>>>
>>>>>>>>>> http://github.com/sgallese/sgallese/tree/master/src/pHashTest/
>>>>>>>>>>
>>>>>>>>>> As you can see from the above code, I have phash.java side by side
>>>>>>>>>> with PHashTest.java in a package caled pHashTest
>>>>>>>>>>
>>>>>>>>>> In PHashTest.java, I just wanted to instantiate pHash, so I have the
>>>>>>>>>> following code:
>>>>>>>>>>
>>>>>>>>>> package pHashTest;
>>>>>>>>>>
>>>>>>>>>> public class PHashTest {
>>>>>>>>>> public static void main(String[] args) {
>>>>>>>>>> pHash mypHash = new pHash();
>>>>>>>>>> }
>>>>>>>>>> }
>>>>>>>>>>
>>>>>>>>>> I kept getting the error "java.lang.UnsatisfiedLinkError: no pHash-jni
>>>>>>>>>> in java.library.path" so I searched for pHash-jni on my system and
>>>>>>>>>> found these:
>>>>>>>>>>
>>>>>>>>>> /usr/local/include/pHash-jni.h
>>>>>>>>>> /usr/local/lib/libpHash-jni.so
>>>>>>>>>> /usr/local/lib/libpHash-jni.so.0
>>>>>>>>>> /usr/local/lib/libpHash-jni.so.0.0.0
>>>>>>>>>> /usr/local/lib/libpHash-jni.la
>>>>>>>>>> /usr/local/lib/libpHash-jni.a
>>>>>>>>>>
>>>>>>>>>> In Eclipse, I then went to run -> run configurations -> Environment
>>>>>>>>>> (tab) and I set LD_LIBRARY_PATH to /usr/local/lib/
>>>>>>>>>>
>>>>>>>>>> Then I tried to initiate a pHash object again (i.e., pHash mypHash =
>>>>>>>>>> new pHash(); ) and I'd get the following error.
>>>>>>>>>>
>>>>>>>>>> Exception in thread "main" java.lang.UnsatisfiedLinkError:
>>>>>>>>>> /usr/local/lib/libpHash-jni.so.0.0.0:
>>>>>>>>>> /usr/local/lib/libpHash-jni.so.0.0.0: undefined symbol:
>>>>>>>>>> _ZL17ph_free_datapointP12ph_datapoint
>>>>>>>>>> at java.lang.ClassLoader$NativeLibrary.load(Native Method)
>>>>>>>>>> at java.lang.ClassLoader.loadLibrary0(ClassLoader.java:1778)
>>>>>>>>>> at java.lang.ClassLoader.loadLibrary(ClassLoader.java:1703)
>>>>>>>>>> at java.lang.Runtime.loadLibrary0(Runtime.java:823)
>>>>>>>>>> at java.lang.System.loadLibrary(System.java:1028)
>>>>>>>>>> at pHashTest.pHash.<clinit>(pHash.java:47)
>>>>>>>>>> at pHashTest.PHashTest.main(PHashTest.java:6)
>>>>>>>>>>
>>>>>>>>>> I figured I'd then try to make an ImageHash object using the code
>>>>>>>>>>
>>>>>>>>>> ImageHash mypHash = new ImageHash("/home/kdge/Pictures/cc.jpg");
>>>>>>>>>>
>>>>>>>>>> but ImageHash would not take a string in its constructor.
>>>>>>>>>>
>>>>>>>>>> Any help is greatly appreciated!
>>>>>>>>>>
>>>>>>>>>> Thanks,
>>>>>>>>>>
>>>>>>>>>> Seb
>>>>>>>>>> _______________________________________________
>>>>>>>>>> pHash-support mailing list
>>>>>>>>>> pHash-support at lists.phash.org
>>>>>>>>>> http://lists.phash.org/listinfo.cgi/phash-support-phash.org
>>>>>>>>>>
>>>>>>>>>
>>>>>>>> _______________________________________________
>>>>>>>> pHash-support mailing list
>>>>>>>> pHash-support at lists.phash.org
>>>>>>>> http://lists.phash.org/listinfo.cgi/phash-support-phash.org
>>>>>>>>
>>>>>>>
>>>>>>
>>>>>
>>>>
>>>
>>
>
More information about the pHash-support
mailing list