search

javaee / javahelp

javahelp
Other
14 stars 15 forks source link

-db does not work with default temp file loc on Mac OS X #33

Closed glassfishrobot closed 12 years ago

glassfishrobot commented 12 years ago

com.sun.java.help.search.Schema.getURL and com.sun.java.help.search.RAFFileFactory.get said to perform improper File <-> URL conversion, producing incorrect results when file names contain + or perhaps other special characters.

Environment

Mac OS X 10.6.8 (javadeveloper_for_mac_os_x_10.6__10m3425.dmg)

glassfishrobot commented 12 years ago

Reported by jglick

glassfishrobot commented 12 years ago

jglick said: DefaultIndexBuilder. is also unsafe.

glassfishrobot commented 12 years ago

jglick said: Test case; works when db's name has no +, but as written throws

java.io.FileNotFoundException: /tmp/jhindexer out4...0/TMAP (No such file or directory)
    at java.io.RandomAccessFile.open(Native Method)
    at java.io.RandomAccessFile.<init>(RandomAccessFile.java:233)
    at java.io.RandomAccessFile.<init>(RandomAccessFile.java:118)
    at com.sun.java.help.search.RAFFile.<init>(RAFFile.java:63)
    at com.sun.java.help.search.RAFFileFactory.get(RAFFileFactory.java:100)
    at com.sun.java.help.search.BlockManager.<init>(BlockManager.java:87)
    at com.sun.java.help.search.BtreeDict.init(BtreeDict.java:212)
    at com.sun.java.help.search.FullBtreeDict.<init>(FullBtreeDict.java:343)
    at com.sun.java.help.search.DefaultIndexBuilder.<init>(DefaultIndexBuilder.java:79)
    at com.sun.java.help.search.Indexer.compile(Indexer.java:197)
    at com.sun.java.help.search.Indexer.main(Indexer.java:86)
    at TestJAVAHELP33.main(TestJAVAHELP33.java:29)
glassfishrobot commented 12 years ago

jglick said: Fixed in revision 89.

glassfishrobot commented 12 years ago

File: TestJAVAHELP33.java Attached By: jglick

glassfishrobot commented 12 years ago

Was assigned to jglick

glassfishrobot commented 7 years ago

This issue was imported from java.net JIRA JAVAHELP-33

glassfishrobot commented 12 years ago

Marked as fixed on Friday, September 30th 2011, 7:52:12 am