Skip to content

The current statement does nothing at all ...#609

Merged
coheigea merged 1 commit intoapache:masterfrom
fabian-barney:patch-1
Dec 4, 2019
Merged

The current statement does nothing at all ...#609
coheigea merged 1 commit intoapache:masterfrom
fabian-barney:patch-1

Conversation

@fabian-barney
Copy link
Contributor

... and it seems to be intended that the ending slash should be removed.

The following code always dows nothing at all:
address = address.substring(0, address.length());

It seems that the ending slash should be removed in some cases:
address = address.substring(0, address.length() - 1);

… nothing at all.

The following code always dows nothing at all:
address = address.substring(0, address.length());

It seems that the ending slash should be removed in some cases:
address = address.substring(0, address.length() - 1);
@coheigea coheigea merged commit e0021c4 into apache:master Dec 4, 2019
coheigea pushed a commit that referenced this pull request Dec 4, 2019
… nothing at all. (#609)

The following code always dows nothing at all:
address = address.substring(0, address.length());

It seems that the ending slash should be removed in some cases:
address = address.substring(0, address.length() - 1);

(cherry picked from commit e0021c4)
coheigea pushed a commit that referenced this pull request Jan 7, 2020
… nothing at all. (#609)

The following code always dows nothing at all:
address = address.substring(0, address.length());

It seems that the ending slash should be removed in some cases:
address = address.substring(0, address.length() - 1);

(cherry picked from commit e0021c4)
(cherry picked from commit bfc1d71)
ppalaga pushed a commit to ppalaga/cxf that referenced this pull request Nov 12, 2024
…ache.maven.plugins-maven-project-info-reports-plugin-3.7.0

Bump org.apache.maven.plugins:maven-project-info-reports-plugin from 3.6.1 to 3.7.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants