[DependencyInjection] made a small cleanup
[DependencyInjection] fixed anonymous services handling in XmlFileLoader

Previous to this commit, it was not possible to have anonymous services
as arguments AND anonymous services in custom configs
[DependencyInjection] added support for anonymous services as properties (closes #2964)
[DomCrawler] converted all usage of filter() to filterXPath() in unit tests to be less dependent on CssSelector
[DomCrawler] fixed encoding when using addHtmlContent() (fixes #3881)
[HttpFoundation] fixed docblock typos in session class
[DependencyInjection] Add ParameterBag::remove
[Routing] Fix a regression introduced by #4170
merged branch vicb/route_compiler (PR #4219)

Commits
-------

9907df2 [Routing] Fix a regression introduced by #4170

Discussion
----------

[Routing] Fix a regression introduced by #4170

See https://github.com/symfony/symfony/commit/8232aa150b70572b1d378f202d886539946853a3#commitcomment-1300310

Let's wait to confirm the fix is ok before merging

@stephpy, @mvrhov ?

(Travis is not ok since 5.3.3 is not supported).

---------------------------------------------------------------------------

by stephpy at 2012-05-07T16:17:44Z

Yes it's OK !

Thanks :)

---------------------------------------------------------------------------

by Tobion at 2012-05-07T16:20:38Z

So your originally proposed spec is not the same anymore.

---------------------------------------------------------------------------

by vicb at 2012-05-07T16:22:38Z

@Tobion

* fixed.
* no it is not as it couldn't work in this case.
merged branch shieldo/session_docblock_typos (PR #4216)

Commits
-------

8ff11c1 [HttpFoundation] fixed docblock typos in session class

Discussion
----------

[HttpFoundation] fixed docblock typos in session class
merged branch Seldaek/parambag (PR #4220)

Commits
-------

3d9990a [DependencyInjection] Add ParameterBag::remove

Discussion
----------

[DependencyInjection] Add ParameterBag::remove
merged branch symfony/domcrawler-encoding (PR #4214)

Commits
-------

c9ebe67 [DomCrawler] fixed encoding when using addHtmlContent() (fixes #3881)

Discussion
----------

[DomCrawler] fixed encoding when using addHtmlContent() (fixes #3881)

After looking around, this is clear that loadHtml() resets the encoding set on the DomDocument instance. So, the only workaround that actually works (and which is not an ugly hack) is to use `mb_convert_encoding` when it exists.

---------------------------------------------------------------------------

by Seldaek at 2012-05-07T12:38:43Z

+1 (Side note: Using your fork of symfony for PRs would be good I think, otherwise it creates noisy versions on packagist.)
Merge remote-tracking branch 'upstream/master'

* upstream/master:
  [Routing] Fix a regression introduced by #4170
  [DependencyInjection] Add ParameterBag::remove
  [HttpFoundation] fixed docblock typos in session class
  [DomCrawler] fixed encoding when using addHtmlContent() (fixes #3881)
  [DomCrawler] converted all usage of filter() to filterXPath() in unit tests to be less dependent on CssSelector
  [DependencyInjection] added support for anonymous services as properties (closes #2964)
  [DependencyInjection] fixed anonymous services handling in XmlFileLoader
  [DependencyInjection] made a small cleanup