+1 for reformatting. I've lived with the terrible code style in taglibs for years because I felt that reformatting it just for me was over the top. Now there are more eyes on the code; PLEASE MAKE IT READABLE! :)
I'm assuming it's a one-time only reformat to get us onto something sane. Hen On Tue, Nov 16, 2010 at 7:45 PM, Rex Wang <[email protected]> wrote: > What's the formatter you plan to use? Is it public and other committer > obeyed? Otherwise, it will mess up again in future.. > > Anyway, +1 for tab replacement with 4 spaces. > > -Rex > > 2010/11/16 Jeremy Boynes <[email protected]> > >> As well as the tabs, there are broader inconsistencies in the style (e.g. >> consistent braces, missing javadoc, and the like) that lead to IDE warnings. >> >> How about running everything through a re-formatter to clean this up? >> Downside is that it will make back-patching harder. >> +1 from me. >> >> On Nov 16, 2010, at 5:28 AM, [email protected] wrote: >> >> > https://issues.apache.org/bugzilla/show_bug.cgi?id=50279 >> > >> > Summary: Tab police >> > Product: Taglibs >> > Version: unspecified >> > Platform: PC >> > OS/Version: Windows XP >> > Status: NEW >> > Severity: normal >> > Priority: P2 >> > Component: Unknown Taglib >> > AssignedTo: [email protected] >> > ReportedBy: [email protected] >> > >> > >> > Created an attachment (id=26301) >> > --> (https://issues.apache.org/bugzilla/attachment.cgi?id=26301) >> > Fix tabs in examples code >> > >> > There are oodles of tabs in the Taglibs code. >> > >> > Tabs seem to be set at 8 spaces, at least in the examples section. >> > >> > I can provide patches for the other code if required. >> > >> > -- >> > Configure bugmail: >> https://issues.apache.org/bugzilla/userprefs.cgi?tab=email >> > ------- You are receiving this mail because: ------- >> > You are the assignee for the bug. >> > >> > --------------------------------------------------------------------- >> > To unsubscribe, e-mail: [email protected] >> > For additional commands, e-mail: [email protected] >> > >> >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: [email protected] >> For additional commands, e-mail: [email protected] >> >> > > > -- > Lei Wang (Rex) > rwonly AT apache.org > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
