Dirk Nehring dixit:

>you delete a description ("remove_debug_honor_prefix")

I've read the name of the patch, then I read the patch itself.
From the name, it was *totally* unclear to me what it meant.

For simple patches like this, additional documentation at the
top of the patches is nonsense¹, but for more complicated patches
you need descriptions anyway.

¹) The filename is not enough, but a 5 second glimpse at
   the patch itself is. Besides, what do you expect from
   a patch to a Makefile? It's always build fixes etc.

//mirabile
-- 
  "Using Lynx is like wearing a really good pair of shades: cuts out
   the glare and harmful UV (ultra-vanity), and you feel so-o-o COOL."
                                         -- Henry Nelson, March 1999
_______________________________________________
freewrt-developers mailing list
[email protected]
https://www.freewrt.org/lists/listinfo/freewrt-developers

Reply via email to