[josm-dev] preparing a patch for #2483

Dirk Stöcker openstreetmap at dstoecker.de
Fri Jun 5 08:50:22 BST 2009


On Thu, 4 Jun 2009, Dieter Muecke wrote:

> I've just created a patch to fix a problem in JOSM  and found lots of
> differences caused by eclipse source code formatter.
> Is it appreciated to send a patch including source code
> beautifications made by eclipse?

No. Do only included format changes, when they apply to the position, 
where you anyway changed stuff. Patches in Trac must be readable and 
formating changes will obscure the real changes.

> What is the next step in trac? Shall I reassign it to another person
> or mark it fixed?

Never mark it fixed. Fixed means "Fixed in SVN". Add a "[PATCH]" in the 
subject, so I see the patch when looking for it. And care for the comments 
you get :-)

Ciao
-- 
http://www.dstoecker.eu/ (PGP key available)





More information about the josm-dev mailing list