Fix a compiler warning in the AVR-COFF generator that appeared with

GCC 4.x, and caused the compilation to fail due to the -Werror option
used.

Remove an annoying assembler warning in the ATmega256x patch that was
frequently triggered by large C switch() statements in GCC, and
generally considered to be a regression.  The author of that patch
agreed that the warning isn't particularly useful anyway for a number
of reasons, and recommended to just drop it.

devel/avr-binutils/Makefile

@@ -7,7 +7,7 @@
 
 PORTNAME=	binutils
 PORTVERSION=	2.17
-PORTREVISION=	2
+PORTREVISION=	3
 CATEGORIES=	devel
 MASTER_SITES=	${MASTER_SITE_SOURCEWARE}
 MASTER_SITE_SUBDIR=	binutils/releases

devel/avr-binutils/files/patch-atmega256x

@@ -1738,7 +1738,7 @@ diff -Nur gas/config/tc-avr.c gas/config/tc-avr.c
 +		      break;
 +
 +		    default:
-+		      as_warn (_("expression dangerous with linker stubs"));
++		      break; /* as_warn (_("expression dangerous with linker stubs")); *//* Bjoern agreed. :) */
 +		    }
 +		}
 +	      return reloc_to_return;

devel/avr-binutils/files/patch-coff-avr

@@ -2811,7 +2811,7 @@
 +      shash->emitted = FALSE;
 +    }
 +  else
-+    coff_struct_hash_lookup (&info->structs, name,
++    (void)coff_struct_hash_lookup (&info->structs, name,
 +			     TRUE, tst->u.ts_struct.tagismalloced);
 +
 +  return TRUE;