I'm unsure. How does one know if that param defines a default value or a requirement? It's too vague.

It's only a default value, not a requirement.
It's just a shortcut to avoiddefaults={"name"="bob"}

Yes, but its not clear. It could also be a shortcut torequirements={"name"="bob"}, which has totally different meaning. So it's not self-explanatory.
-1 for me.

it is the default php behavior. It's a default value for a variable...

@Tobion using the default value of the method to set a requirement does not make any sense. I don't see why someone would expect this behavior

@lyrixx Can you add some unit tests?

Oh I misunderstood the PR. I thought this makes thename param default tohi.@Route("/hi/{name}", name="hi"). But it's just the name of the route. Your example was easy to misinterpret as you usedname everywhere.

@lyrixx Can you finish this PR?

@fabpot Yes i will as soon as possible.

I rebase and amend my commit. (I changed doc in commit message to be less confusing)

I will try to add tests.
But for now,AnnotationClassLoader::load is not really tested, andAnnotationClassLoader::addRoute is absolutely not tested. So I think I should add tests for these methods ? And then add tests for my patch.
I will try tomorrow.

@fabpot I added new tests. I tried to made very atomic commits.