[
https://issues.apache.org/jira/browse/JEXL-157?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14608077#comment-14608077
]
Sebb edited comment on JEXL-157 at 6/30/15 10:19 AM:
-----------------------------------------------------
Note: it was probably not spotted because System.getProperty("java.io.tmpdir")
returns a string ending with "/" - i.e. the separator char - (at least on my
system) and pathSeparator (\:) is valid as a leading filename character on many
systems.
I think it's safer to use the appropriate File class ctor rather than making
assumptions about how OS file path names are constructed.
was (Author: s...@apache.org):
Note: it was probably not spotted because System.getProperty("java.io.tmpdir")
returns a string ending with "/" - i.e. the separator char - (at least on my
system) and pathSeparator (:) is valid as a leading filename character on many
systems.
I think it's safer to use the appropriate File class ctor rather than making
assumptions about how OS file path names are constructed.
> Replace File.pathSeparator with File.separator
> ----------------------------------------------
>
> Key: JEXL-157
> URL: https://issues.apache.org/jira/browse/JEXL-157
> Project: Commons JEXL
> Issue Type: Bug
> Affects Versions: 3.0
> Environment: Debian Jessie 8.1 64 Bit
> Maven 3.0.5-3
> OpenJDK 7u79-2.5.5-1~deb8u1
> German language
> Reporter: Lars Cebulla
> Assignee: Henri Biestro
> Priority: Blocker
>
> Hi,
> I got errors when I try to compile / test JEXL because ClassCreatorTest fails.
> At line 51, it uses "File.pathSeparator", which is wrong. It has to be just
> "File.separator".
> Thanks in advance!
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
