[jbossseam-issues] [JBoss JIRA] Closed: (JBSEAM-1186) log API improvement
Gavin King (JIRA)
jira-events at lists.jboss.org
Mon Jun 11 01:13:11 EDT 2007
[ http://jira.jboss.com/jira/browse/JBSEAM-1186?page=all ]
Gavin King closed JBSEAM-1186.
------------------------------
Resolution: Rejected
#2 #1 #3 is allowed
> log API improvement
> -------------------
>
> Key: JBSEAM-1186
> URL: http://jira.jboss.com/jira/browse/JBSEAM-1186
> Project: JBoss Seam
> Issue Type: Feature Request
> Components: Core
> Environment: all
> Reporter: Mikhail Grushinskiy
> Priority: Minor
>
> Instead of
> log.debug("Creating new order for user: #0 product: #1 quantity: #2", user.username(), product.name(), quantity);
> #0, #1, #2
> why couldn't we just always use
> #?
> Parameters are already ordered. There is no need to number them in the format string, this will be just another source of errors.
> Another question what happens if you have more parameters after format string than
> #n placeholders. I think in such case these parameters should be appended
> at the end of the format string in output so not values are lost
> .
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
More information about the seam-issues
mailing list