[DomCrawler] Added missing docblocks and removed unneeded return annotation.
[Translation] Removed an unneeded return annotation.
merged branch jakzal/docblock-cleanup (PR #9028)

This PR was merged into the 2.2 branch.

Discussion
----------

[Translation] Removed a @return annotation

This is the only `@return null` left...

Commits
-------

d6f4def [Translation] Removed an unneeded return annotation.
merged branch jakzal/domcrawler-missing-docblocks (PR #9025)

This PR was merged into the 2.2 branch.

Discussion
----------

[DomCrawler] Added missing docblocks and removed redundant type in a return annotation

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

Commits
-------

d414213 [DomCrawler] Added missing docblocks and removed unneeded return annotation.
[Form] Fixed: "required" attribute is not added to <select> tag if no empty value
merged branch bschussek/issue8942 (PR #9030)

This PR was merged into the 2.3 branch.

Discussion
----------

[Form] Fixed: "required" attribute is not added to <select> tag if no empty value

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #8942
| License       | MIT
| Doc PR        | -

Commits
-------

a273e79 [Form] Fixed: "required" attribute is not added to <select> tag if no empty value
added missing method in the UPGRADE file for 2.2 (closes #8941)
LICENSE files moved to meta folders
merged branch peterkokot/patch-license (PR #9041)

This PR was submitted for the 2.1 branch but it was merged into the 2.2 branch instead (closes #9041).

Discussion
----------

Patch license

According to the documentation: http://symfony.com/doc/current/cookbook/bundles/best_practices.html#directory-structure I think LICENSE files should be in meta folders in Symfony core bundles as well. If I'm mistaken you can close this pull request. Thanks.

Commits
-------

66bafd2 LICENSE files moved to meta folders
HttpFoundation Request test for #8619
HttpFoundation RequestTest - Fixed indentation and removed comments
merged branch fturmel/patch-1 (PR #9038)

This PR was submitted for the 2.3 branch but it was merged into the 2.2 branch instead (closes #9038).

Discussion
----------

HttpFoundation Request : unit test to confirm #8619 issue fix

Commits
-------

ae4925b HttpFoundation RequestTest - Fixed indentation and removed comments
94ea510 HttpFoundation Request test for #8619
Typo fix

Small typo/grammar fix in class' comment
merged branch sarcasme/patch-1 (PR #9037)

This PR was submitted for the master branch but it was merged into the 2.2 branch instead (closes #9037).

Discussion
----------

Typo fix

Small typo/grammar fix in class' comment

Commits
-------

16f455e Typo fix
fixed bytes conversion when used on 32-bits systems
merged branch fabpot/bytes-conversion-epic-fail (PR #9048)

This PR was merged into the 2.3 branch.

Discussion
----------

fixed bytes conversion when used on 32-bits systems

| Q             | A
| ------------- | ---
| Bug fix?      | yes (on 32-bits systems)
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #8977
| License       | MIT
| Doc PR        | n/a

This PR reverts #7413 and #742, which does not work well when a number is big (3Go for instance) and the machine is 32bits.

Commits
-------

b3ae29d fixed bytes conversion when used on 32-bits systems
[Process] Properly close pipes after a Process::stop call
merged branch romainneutron/timeout-restart (PR #9050)

This PR was merged into the 2.3 branch.

Discussion
----------

[Process][2.3] Properly close pipes after a Process::stop call

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| License       | MIT

When calling `Process::stop`, if the process does not stop before the end of the method, pipes are not close, `proc_close` is not called.
I added a test that randomly fails without the patch :

```
phpunit --filter=testStartAfterATimeout --repeat 100 Tests/SimpleProcessTest.php
PHPUnit 3.8-g55a6dd0 by Sebastian Bergmann.

Configuration read from /Users/romain/Documents/workspace/symfony/src/Symfony/Component/Process/phpunit.xml.dist

....E.E......E.....E..........EE..............EEE..EE.......E..  63 / 100 ( 63%)
E.E................E....EE.......E...

Time: 29.55 seconds, Memory: 6.75Mb

There were 18 errors:

1) Symfony\Component\Process\Tests\SimpleProcessTest::testStartAfterATimeout
fclose(): 89 is not a valid stream resource

/Users/romain/Documents/workspace/symfony/src/Symfony/Component/Process/ProcessPipes.php:95
/Users/romain/Documents/workspace/symfony/src/Symfony/Component/Process/ProcessPipes.php:80
/Users/romain/Documents/workspace/symfony/src/Symfony/Component/Process/ProcessPipes.php:62
/Users/romain/Documents/workspace/symfony/src/Symfony/Component/Process/Process.php:938
/Users/romain/Documents/workspace/symfony/src/Symfony/Component/Process/Process.php:229
/Users/romain/Documents/workspace/symfony/src/Symfony/Component/Process/Tests/AbstractProcessTest.php:490
```

And of course, I solved the issue, tests are now OK.

Commits
-------

d84df4c [Process] Properly close pipes after a Process::stop call
[BrowserKit] Fixed the handling of parameters when redirecting

POST parameters should not be transmitted as GET parameters after the
redirection when changing the method.
Making tests pass on mac os x without this change tests would fail under mac os x at least in 10.8.2