Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix circular placeholder prevention #42

Closed
wants to merge 1 commit into from
Closed

Fix circular placeholder prevention #42

wants to merge 1 commit into from

Conversation

sslavic
Copy link
Contributor

@sslavic sslavic commented Feb 18, 2012

Set of resolved placeholder references is used for circular placeholder
prevention. For complex property definitions this mechanism would put
property values with unresolved inner placeholder references in the
set, but would try to remove property values with placeholders
resolved, leaving set in invalid state and mechanism broken.

This fix makes sure value that is put in set is same one that is
removed from it, and by this avoid false positives in reporting
circular placeholder.

Issue: SPR-5369

I have signed and agree to the terms in the SpringSource Individual Contributor License Agreement.

Set of resolved placeholder references is used for circular placeholder
prevention. For complex property definitions this mechanism would put
property values with unresolved inner placeholder references in the
set, but would try to remove property values with placeholders
resolved, leaving set in invalid state and mechanism broken.

This fix makes sure value that is put in set is same one that is
removed from it, and by this avoid false positives in reporting
circular placeholder.

Issue: SPR-5369
cbeams added a commit that referenced this pull request May 17, 2012
* SPR-5369:
  Fix circular placeholder prevention
@cbeams
Copy link
Contributor

cbeams commented May 17, 2012

Thanks, Stevo! Committed, after changing Apache license date ranges and touching up the commit comment for minor grammar issues.

@cbeams cbeams closed this May 17, 2012
GreenRecycleBin added a commit to GreenRecycleBin/spring-framework that referenced this pull request Aug 9, 2019
"
{@code}

    {@code text }

Equivalent to <code>{@literal text }</code>.

Displays text in code font without interpreting the text as HTML markup or nested Javadoc tags.
"
https://docs.oracle.com/en/java/javase/12/docs/specs/doc-comment-spec.html

This is why {@code &spring-projects#42;&spring-projects#47;&spring-projects#42;} isn't rendered as <code>*/*</code> as expected.
sbrannen pushed a commit that referenced this pull request Aug 9, 2019
This commit changes {@code &#42;&#47;&#42;} to <code>&#42;&#47;&#42;</code>
so that "*/*" is rendered correctly.

Closes gh-23439
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