From cdb4a3568d68bbb834051473cb9485f87bc61b16 Mon Sep 17 00:00:00 2001
From: Conduitry <git@fur.army>
Date: Tue, 20 Dec 2016 23:21:28 -0500
Subject: [PATCH 1/4] use empty comments as anchors

---
 src/generators/dom/index.js              | 4 ++--
 src/generators/dom/visitors/EachBlock.js | 2 +-
 src/generators/dom/visitors/IfBlock.js   | 2 +-
 src/generators/dom/visitors/YieldTag.js  | 2 +-
 src/shared/dom.js                        | 4 ++--
 5 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/generators/dom/index.js b/src/generators/dom/index.js
index 929d737700..de0c5dc496 100644
--- a/src/generators/dom/index.js
+++ b/src/generators/dom/index.js
@@ -98,9 +98,9 @@ class DomGenerator extends Generator {
 		` );
 	}
 
-	createAnchor ( name, description = '' ) {
+	createAnchor ( name ) {
 		this.uses.createComment = true;
-		const renderStatement = `createComment( ${JSON.stringify( description )} )`;
+		const renderStatement = `createComment()`;
 		this.addElement( name, renderStatement, true );
 	}
 
diff --git a/src/generators/dom/visitors/EachBlock.js b/src/generators/dom/visitors/EachBlock.js
index 2799a421b8..700e16ef7a 100644
--- a/src/generators/dom/visitors/EachBlock.js
+++ b/src/generators/dom/visitors/EachBlock.js
@@ -21,7 +21,7 @@ export default {
 		const { dependencies, snippet } = generator.contextualise( node.expression );
 
 		const anchor = `${name}_anchor`;
-		generator.createAnchor( anchor, `#each ${generator.source.slice( node.expression.start, node.expression.end )}` );
+		generator.createAnchor( anchor );
 
 		generator.current.builders.init.addLine( `var ${name}_value = ${snippet};` );
 		generator.current.builders.init.addLine( `var ${iterations} = [];` );
diff --git a/src/generators/dom/visitors/IfBlock.js b/src/generators/dom/visitors/IfBlock.js
index 02133e539d..5ceb42d48b 100644
--- a/src/generators/dom/visitors/IfBlock.js
+++ b/src/generators/dom/visitors/IfBlock.js
@@ -41,7 +41,7 @@ export default {
 		const conditionsAndBlocks = getConditionsAndBlocks( generator, node, generator.getUniqueName( `renderIfBlock` ) );
 
 		const anchor = `${name}_anchor`;
-		generator.createAnchor( anchor, `#if ${generator.source.slice( node.expression.start, node.expression.end )}` );
+		generator.createAnchor( anchor );
 
 		generator.current.builders.init.addBlock( deindent`
 			function ${getBlock} ( ${params} ) {
diff --git a/src/generators/dom/visitors/YieldTag.js b/src/generators/dom/visitors/YieldTag.js
index 5fefcd73a3..98996486c1 100644
--- a/src/generators/dom/visitors/YieldTag.js
+++ b/src/generators/dom/visitors/YieldTag.js
@@ -1,7 +1,7 @@
 export default {
 	enter ( generator ) {
 		const anchor = `yield_anchor`;
-		generator.createAnchor( anchor, 'yield' );
+		generator.createAnchor( anchor );
 
 		generator.current.builders.mount.addLine(
 			`component._yield && component._yield.mount( ${generator.current.target}, ${anchor} );`
diff --git a/src/shared/dom.js b/src/shared/dom.js
index 9d4231572f..4007e90289 100644
--- a/src/shared/dom.js
+++ b/src/shared/dom.js
@@ -34,8 +34,8 @@ export function createText ( data ) {
 	return document.createTextNode( data );
 }
 
-export function createComment ( data ) {
-	return document.createComment( data );
+export function createComment () {
+	return document.createComment( '' );
 }
 
 export function addEventListener ( node, event, handler ) {

From 2316d7d3482ca3f31dfe3e0fe5cfd9eee9463364 Mon Sep 17 00:00:00 2001
From: Conduitry <git@fur.army>
Date: Tue, 27 Dec 2016 20:59:43 -0500
Subject: [PATCH 2/4] fix unit tests

---
 test/generator/attribute-dynamic-multiple/_config.js        | 2 +-
 test/generator/binding-input-text-contextual/_config.js     | 6 +++---
 .../generator/binding-input-text-deep-contextual/_config.js | 6 +++---
 test/generator/component-yield-parent/_config.js            | 6 +++---
 test/generator/each-block-indexed/_config.js                | 2 +-
 test/generator/each-block/_config.js                        | 6 +++---
 test/generator/each-blocks-expression/_config.js            | 2 +-
 test/generator/each-blocks-nested-b/_config.js              | 2 +-
 test/generator/each-blocks-nested/_config.js                | 2 +-
 test/generator/event-handler-removal/_config.js             | 2 +-
 test/generator/event-handler/_config.js                     | 6 +++---
 test/generator/if-block-widget/_config.js                   | 6 +++---
 test/generator/if-block/_config.js                          | 6 +++---
 13 files changed, 27 insertions(+), 27 deletions(-)

diff --git a/test/generator/attribute-dynamic-multiple/_config.js b/test/generator/attribute-dynamic-multiple/_config.js
index 83c6a58a86..1a0c602cb6 100644
--- a/test/generator/attribute-dynamic-multiple/_config.js
+++ b/test/generator/attribute-dynamic-multiple/_config.js
@@ -7,5 +7,5 @@ export default {
 		]
 	},
 
-	html: `<div class="foo ">1</div><div class=" bar">2</div><div class="foo bar">3</div><!--#each items-->`
+	html: `<div class="foo ">1</div><div class=" bar">2</div><div class="foo bar">3</div><!---->`
 };
diff --git a/test/generator/binding-input-text-contextual/_config.js b/test/generator/binding-input-text-contextual/_config.js
index 3f2d3f3fc0..332d2c6f39 100644
--- a/test/generator/binding-input-text-contextual/_config.js
+++ b/test/generator/binding-input-text-contextual/_config.js
@@ -6,7 +6,7 @@ export default {
 			'three'
 		]
 	},
-	html: `<div><input><p>one</p></div><div><input><p>two</p></div><div><input><p>three</p></div><!--#each items-->`,
+	html: `<div><input><p>one</p></div><div><input><p>two</p></div><div><input><p>three</p></div><!---->`,
 	test ( assert, component, target, window ) {
 		const inputs = [ ...target.querySelectorAll( 'input' ) ];
 		const items = component.get( 'items' );
@@ -18,12 +18,12 @@ export default {
 		inputs[1].dispatchEvent( event );
 
 		assert.equal( items[1], 'four' );
-		assert.equal( target.innerHTML, `<div><input><p>one</p></div><div><input><p>four</p></div><div><input><p>three</p></div><!--#each items-->` );
+		assert.equal( target.innerHTML, `<div><input><p>one</p></div><div><input><p>four</p></div><div><input><p>three</p></div><!---->` );
 
 		items[2] = 'five';
 
 		component.set({ items });
 		assert.equal( inputs[2].value, 'five' );
-		assert.equal( target.innerHTML, `<div><input><p>one</p></div><div><input><p>four</p></div><div><input><p>five</p></div><!--#each items-->` );
+		assert.equal( target.innerHTML, `<div><input><p>one</p></div><div><input><p>four</p></div><div><input><p>five</p></div><!---->` );
 	}
 };
diff --git a/test/generator/binding-input-text-deep-contextual/_config.js b/test/generator/binding-input-text-deep-contextual/_config.js
index f24902a962..587110786c 100644
--- a/test/generator/binding-input-text-deep-contextual/_config.js
+++ b/test/generator/binding-input-text-deep-contextual/_config.js
@@ -6,7 +6,7 @@ export default {
 			{ description: 'three' }
 		]
 	},
-	html: `<div><input><p>one</p></div><div><input><p>two</p></div><div><input><p>three</p></div><!--#each items-->`,
+	html: `<div><input><p>one</p></div><div><input><p>two</p></div><div><input><p>three</p></div><!---->`,
 	test ( assert, component, target, window ) {
 		const inputs = [ ...target.querySelectorAll( 'input' ) ];
 
@@ -17,13 +17,13 @@ export default {
 		inputs[1].value = 'four';
 		inputs[1].dispatchEvent( event );
 
-		assert.equal( target.innerHTML, `<div><input><p>one</p></div><div><input><p>four</p></div><div><input><p>three</p></div><!--#each items-->` );
+		assert.equal( target.innerHTML, `<div><input><p>one</p></div><div><input><p>four</p></div><div><input><p>three</p></div><!---->` );
 
 		const items = component.get( 'items' );
 		items[2].description = 'five';
 
 		component.set({ items });
 		assert.equal( inputs[2].value, 'five' );
-		assert.equal( target.innerHTML, `<div><input><p>one</p></div><div><input><p>four</p></div><div><input><p>five</p></div><!--#each items-->` );
+		assert.equal( target.innerHTML, `<div><input><p>one</p></div><div><input><p>four</p></div><div><input><p>five</p></div><!---->` );
 	}
 };
diff --git a/test/generator/component-yield-parent/_config.js b/test/generator/component-yield-parent/_config.js
index 28b8e175de..cc3b39ca81 100644
--- a/test/generator/component-yield-parent/_config.js
+++ b/test/generator/component-yield-parent/_config.js
@@ -2,8 +2,8 @@ export default {
 	html: '<div><p>Hello</p></div>',
 	test ( assert, component, target ) {
 		assert.equal( component.get( 'data' ), 'Hello' );
-		component.set({data: 'World'})
+		component.set({data: 'World'});
 		assert.equal( component.get( 'data' ), 'World' );
-		assert.equal( target.innerHTML, '<div><p>World<!--yield--></p></div>' );
+		assert.equal( target.innerHTML, '<div><p>World<!----></p></div>' );
 	}
-}
+};
diff --git a/test/generator/each-block-indexed/_config.js b/test/generator/each-block-indexed/_config.js
index 34640e18cb..5aaadafb05 100644
--- a/test/generator/each-block-indexed/_config.js
+++ b/test/generator/each-block-indexed/_config.js
@@ -2,5 +2,5 @@ export default {
 	data: {
 		animals: [ 'adder', 'blue whale', 'chameleon' ]
 	},
-	html: `<p>0: adder</p><p>1: blue whale</p><p>2: chameleon</p><!--#each animals-->`
+	html: `<p>0: adder</p><p>1: blue whale</p><p>2: chameleon</p><!---->`
 };
diff --git a/test/generator/each-block/_config.js b/test/generator/each-block/_config.js
index 959bf88c04..655556bedf 100644
--- a/test/generator/each-block/_config.js
+++ b/test/generator/each-block/_config.js
@@ -2,11 +2,11 @@ export default {
 	data: {
 		animals: [ 'alpaca', 'baboon', 'capybara' ]
 	},
-	html: '<p>alpaca</p><p>baboon</p><p>capybara</p><!--#each animals-->',
+	html: '<p>alpaca</p><p>baboon</p><p>capybara</p><!---->',
 	test ( assert, component, target ) {
 		component.set({ animals: [ 'alpaca', 'baboon', 'caribou', 'dogfish' ] });
-		assert.equal( target.innerHTML, '<p>alpaca</p><p>baboon</p><p>caribou</p><p>dogfish</p><!--#each animals-->' );
+		assert.equal( target.innerHTML, '<p>alpaca</p><p>baboon</p><p>caribou</p><p>dogfish</p><!---->' );
 		component.set({ animals: [] });
-		assert.equal( target.innerHTML, '<!--#each animals-->' );
+		assert.equal( target.innerHTML, '<!---->' );
 	}
 };
diff --git a/test/generator/each-blocks-expression/_config.js b/test/generator/each-blocks-expression/_config.js
index b610b055b4..239a18b6dc 100644
--- a/test/generator/each-blocks-expression/_config.js
+++ b/test/generator/each-blocks-expression/_config.js
@@ -1,3 +1,3 @@
 export default {
-	html: `<p>a</p><p>b</p><p>c</p><!--#each [ 'a', 'b', 'c' ]-->`
+	html: `<p>a</p><p>b</p><p>c</p><!---->`
 };
diff --git a/test/generator/each-blocks-nested-b/_config.js b/test/generator/each-blocks-nested-b/_config.js
index 806284ef2d..1ba0f685b8 100644
--- a/test/generator/each-blocks-nested-b/_config.js
+++ b/test/generator/each-blocks-nested-b/_config.js
@@ -28,7 +28,7 @@ export default {
 			}
 		]
 	},
-	html: `<p>animals: aardvark</p><p>animals: buffalo</p><p>animals: chinchilla</p><!--#each category.things--><p>countries: albania</p><p>countries: brazil</p><p>countries: china</p><!--#each category.things--><p>people: alice</p><p>people: bob</p><p>people: carol</p><p>people: dave</p><!--#each category.things--><!--#each categories-->`,
+	html: `<p>animals: aardvark</p><p>animals: buffalo</p><p>animals: chinchilla</p><!----><p>countries: albania</p><p>countries: brazil</p><p>countries: china</p><!----><p>people: alice</p><p>people: bob</p><p>people: carol</p><p>people: dave</p><!----><!---->`,
 	test ( assert, component, target ) {
 		// TODO
 	}
diff --git a/test/generator/each-blocks-nested/_config.js b/test/generator/each-blocks-nested/_config.js
index 0c21561e75..5486c8274e 100644
--- a/test/generator/each-blocks-nested/_config.js
+++ b/test/generator/each-blocks-nested/_config.js
@@ -4,7 +4,7 @@ export default {
 		rows: [ 1, 2, 3 ]
 	},
 
-	html: `<div>a, 1</div><div>a, 2</div><div>a, 3</div><!--#each rows--><div>b, 1</div><div>b, 2</div><div>b, 3</div><!--#each rows--><div>c, 1</div><div>c, 2</div><div>c, 3</div><!--#each rows--><!--#each columns-->`,
+	html: `<div>a, 1</div><div>a, 2</div><div>a, 3</div><!----><div>b, 1</div><div>b, 2</div><div>b, 3</div><!----><div>c, 1</div><div>c, 2</div><div>c, 3</div><!----><!---->`,
 
 	test ( assert, component, target ) {
 		// TODO
diff --git a/test/generator/event-handler-removal/_config.js b/test/generator/event-handler-removal/_config.js
index 91bc0452a5..b196db1765 100644
--- a/test/generator/event-handler-removal/_config.js
+++ b/test/generator/event-handler-removal/_config.js
@@ -1,7 +1,7 @@
 // TODO gah, JSDOM appears to behave differently to real browsers here... probably need to raise an issue
 
 export default {
-	html: '<input><!--#if visible-->',
+	html: '<input><!---->',
 	test ( assert, component ) {
 		component.refs.input.focus();
 
diff --git a/test/generator/event-handler/_config.js b/test/generator/event-handler/_config.js
index 5fb7f6bc86..8aa60ff387 100644
--- a/test/generator/event-handler/_config.js
+++ b/test/generator/event-handler/_config.js
@@ -1,13 +1,13 @@
 export default {
-	html: '<button>toggle</button>\n\n<!--#if visible-->',
+	html: '<button>toggle</button>\n\n<!---->',
 	test ( assert, component, target, window ) {
 		const button = target.querySelector( 'button' );
 		const event = new window.MouseEvent( 'click' );
 
 		button.dispatchEvent( event );
-		assert.equal( target.innerHTML, '<button>toggle</button>\n\n<p>hello!</p><!--#if visible-->' );
+		assert.equal( target.innerHTML, '<button>toggle</button>\n\n<p>hello!</p><!---->' );
 
 		button.dispatchEvent( event );
-		assert.equal( target.innerHTML, '<button>toggle</button>\n\n<!--#if visible-->' );
+		assert.equal( target.innerHTML, '<button>toggle</button>\n\n<!---->' );
 	}
 };
diff --git a/test/generator/if-block-widget/_config.js b/test/generator/if-block-widget/_config.js
index 674fefb0ba..40d9d050e9 100644
--- a/test/generator/if-block-widget/_config.js
+++ b/test/generator/if-block-widget/_config.js
@@ -2,11 +2,11 @@ export default {
 	data: {
 		visible: true
 	},
-	html: 'before\n<p>Widget</p><!--#if visible-->\nafter',
+	html: 'before\n<p>Widget</p><!---->\nafter',
 	test ( assert, component, target ) {
 		component.set({ visible: false });
-		assert.equal( target.innerHTML, 'before\n<!--#if visible-->\nafter' );
+		assert.equal( target.innerHTML, 'before\n<!---->\nafter' );
 		component.set({ visible: true });
-		assert.equal( target.innerHTML, 'before\n<p>Widget</p><!--#if visible-->\nafter' );
+		assert.equal( target.innerHTML, 'before\n<p>Widget</p><!---->\nafter' );
 	}
 };
diff --git a/test/generator/if-block/_config.js b/test/generator/if-block/_config.js
index f8014c5c9a..4194fd29bc 100644
--- a/test/generator/if-block/_config.js
+++ b/test/generator/if-block/_config.js
@@ -2,11 +2,11 @@ export default {
 	data: {
 		visible: true
 	},
-	html: '<p>i am visible</p><!--#if visible-->',
+	html: '<p>i am visible</p><!---->',
 	test ( assert, component, target ) {
 		component.set({ visible: false });
-		assert.equal( target.innerHTML, '<!--#if visible-->' );
+		assert.equal( target.innerHTML, '<!---->' );
 		component.set({ visible: true });
-		assert.equal( target.innerHTML, '<p>i am visible</p><!--#if visible-->' );
+		assert.equal( target.innerHTML, '<p>i am visible</p><!---->' );
 	}
 };

From fa705302616e511860401b98df76426e687be1c0 Mon Sep 17 00:00:00 2001
From: Rich Harris <richard.a.harris@gmail.com>
Date: Tue, 24 Jan 2017 17:03:14 -0500
Subject: [PATCH 3/4] handle xlink attributes

---
 .../attributes/addElementAttributes.js        | 51 +++++++++++--------
 src/shared/dom.js                             |  4 ++
 test/generator/svg-xlink/_config.js           | 19 +++++++
 test/generator/svg-xlink/main.html            |  7 +++
 test/helpers.js                               |  4 ++
 5 files changed, 64 insertions(+), 21 deletions(-)
 create mode 100644 test/generator/svg-xlink/_config.js
 create mode 100644 test/generator/svg-xlink/main.html

diff --git a/src/generators/dom/visitors/attributes/addElementAttributes.js b/src/generators/dom/visitors/attributes/addElementAttributes.js
index 490d330348..c6009ff2e8 100644
--- a/src/generators/dom/visitors/attributes/addElementAttributes.js
+++ b/src/generators/dom/visitors/attributes/addElementAttributes.js
@@ -5,15 +5,24 @@ import flattenReference from '../../../../utils/flattenReference.js';
 
 export default function addElementAttributes ( generator, node, local ) {
 	node.attributes.forEach( attribute => {
+		const name = attribute.name;
+
 		if ( attribute.type === 'Attribute' ) {
-			let metadata = local.namespace ? null : attributeLookup[ attribute.name ];
+			let metadata = local.namespace ? null : attributeLookup[ name ];
 			if ( metadata && metadata.appliesTo && !~metadata.appliesTo.indexOf( node.name ) ) metadata = null;
 
 			let dynamic = false;
 
-			const isBoundOptionValue = node.name === 'option' && attribute.name === 'value'; // TODO check it's actually bound
+			const isBoundOptionValue = node.name === 'option' && name === 'value'; // TODO check it's actually bound
 			const propertyName = isBoundOptionValue ? '__value' : metadata && metadata.propertyName;
 
+			const isXlink = name.slice( 0, 6 ) === 'xlink:';
+
+			// xlink is a special case... we could maybe extend this to generic
+			// namespaced attributes but I'm not sure that's applicable in
+			// HTML5?
+			const helper = isXlink ? 'setXlinkAttribute' : 'setAttribute';
+
 			if ( attribute.value === true ) {
 				// attributes without values, e.g. <textarea readonly>
 				if ( propertyName ) {
@@ -21,14 +30,14 @@ export default function addElementAttributes ( generator, node, local ) {
 						`${local.name}.${propertyName} = true;`
 					);
 				} else {
-					generator.uses.setAttribute = true;
+					generator.uses[ helper ] = true;
 					local.init.addLine(
-						`setAttribute( ${local.name}, '${attribute.name}', true );`
+						`${helper}( ${local.name}, '${name}', true );`
 					);
 				}
 
 				// special case – autofocus. has to be handled in a bit of a weird way
-				if ( attribute.name === 'autofocus' ) {
+				if ( name === 'autofocus' ) {
 					generator.current.autofocus = local.name;
 				}
 			}
@@ -39,9 +48,9 @@ export default function addElementAttributes ( generator, node, local ) {
 						`${local.name}.${propertyName} = '';`
 					);
 				} else {
-					generator.uses.setAttribute = true;
+					generator.uses[ helper ] = true;
 					local.init.addLine(
-						`setAttribute( ${local.name}, '${attribute.name}', '' );`
+						`${helper}( ${local.name}, '${name}', '' );`
 					);
 				}
 			}
@@ -55,7 +64,7 @@ export default function addElementAttributes ( generator, node, local ) {
 					// static attributes
 					result = JSON.stringify( value.data );
 
-					if ( attribute.name === 'xmlns' ) {
+					if ( name === 'xmlns' ) {
 						// special case
 						// TODO this attribute must be static – enforce at compile time
 						local.namespace = value.data;
@@ -64,9 +73,9 @@ export default function addElementAttributes ( generator, node, local ) {
 							`${local.name}.${propertyName} = ${result};`
 						);
 					} else {
-						generator.uses.setAttribute = true;
+						generator.uses[ helper ] = true;
 						local.init.addLine(
-							`setAttribute( ${local.name}, '${attribute.name}', ${result} );`
+							`${helper}( ${local.name}, '${name}', ${result} );`
 						);
 					}
 				}
@@ -81,8 +90,8 @@ export default function addElementAttributes ( generator, node, local ) {
 					if (propertyName) {
 						updater = `${local.name}.${propertyName} = ${snippet};`;
 					} else {
-						generator.uses.setAttribute = true;
-						updater = `setAttribute( ${local.name}, '${attribute.name}', ${snippet} );`; // TODO use snippet both times – see note below
+						generator.uses[ helper ] = true;
+						updater = `${helper}( ${local.name}, '${name}', ${snippet} );`; // TODO use snippet both times – see note below
 					}
 
 					local.init.addLine( updater );
@@ -110,8 +119,8 @@ export default function addElementAttributes ( generator, node, local ) {
 				if (propertyName) {
 					updater = `${local.name}.${propertyName} = ${value};`;
 				} else {
-					generator.uses.setAttribute = true;
-					updater = `setAttribute( ${local.name}, '${attribute.name}', ${value} );`;
+					generator.uses[ helper ] = true;
+					updater = `${helper}( ${local.name}, '${name}', ${value} );`;
 				}
 
 				local.init.addLine( updater );
@@ -153,12 +162,12 @@ export default function addElementAttributes ( generator, node, local ) {
 				return `var ${listName} = this.__svelte.${listName}, ${indexName} = this.__svelte.${indexName}, ${name} = ${listName}[${indexName}]`;
 			});
 
-			const handlerName = generator.current.getUniqueName( `${attribute.name}Handler` );
+			const handlerName = generator.current.getUniqueName( `${name}Handler` );
 			const handlerBody = ( declarations.length ? declarations.join( '\n' ) + '\n\n' : '' ) + `[✂${attribute.expression.start}-${attribute.expression.end}✂];`;
 
-			if ( attribute.name in generator.events ) {
+			if ( name in generator.events ) {
 				local.init.addBlock( deindent`
-					var ${handlerName} = template.events.${attribute.name}.call( component, ${local.name}, function ( event ) {
+					var ${handlerName} = template.events.${name}.call( component, ${local.name}, function ( event ) {
 						${handlerBody}
 					});
 				` );
@@ -174,11 +183,11 @@ export default function addElementAttributes ( generator, node, local ) {
 						${handlerBody}
 					}
 
-					addEventListener( ${local.name}, '${attribute.name}', ${handlerName} );
+					addEventListener( ${local.name}, '${name}', ${handlerName} );
 				` );
 
 				generator.current.builders.teardown.addLine( deindent`
-					removeEventListener( ${local.name}, '${attribute.name}', ${handlerName} );
+					removeEventListener( ${local.name}, '${name}', ${handlerName} );
 				` );
 			}
 		}
@@ -191,11 +200,11 @@ export default function addElementAttributes ( generator, node, local ) {
 			generator.usesRefs = true;
 
 			local.init.addLine(
-				`component.refs.${attribute.name} = ${local.name};`
+				`component.refs.${name} = ${local.name};`
 			);
 
 			generator.current.builders.teardown.addLine( deindent`
-				if ( component.refs.${attribute.name} === ${local.name} ) component.refs.${attribute.name} = null;
+				if ( component.refs.${name} === ${local.name} ) component.refs.${name} = null;
 			` );
 		}
 
diff --git a/src/shared/dom.js b/src/shared/dom.js
index 9d4231572f..62cedcd467 100644
--- a/src/shared/dom.js
+++ b/src/shared/dom.js
@@ -49,3 +49,7 @@ export function removeEventListener ( node, event, handler ) {
 export function setAttribute ( node, attribute, value ) {
 	node.setAttribute ( attribute, value );
 }
+
+export function setXlinkAttribute ( node, attribute, value ) {
+	node.setAttributeNS( 'http://www.w3.org/1999/xlink', attribute, value );
+}
diff --git a/test/generator/svg-xlink/_config.js b/test/generator/svg-xlink/_config.js
new file mode 100644
index 0000000000..eb49f3a72a
--- /dev/null
+++ b/test/generator/svg-xlink/_config.js
@@ -0,0 +1,19 @@
+export default {
+	html: `
+		<svg>
+			<defs>
+				<circle id='stamp' r='10' fill='blue'/>
+			</defs>
+
+			<use xlink:href='#stamp' x='20' y='20'/>
+		</svg>
+	`,
+	test ( assert, component, target ) {
+		const use = target.querySelector( 'use' );
+		const href = use.attributes[ 'xlink:href' ];
+
+		assert.equal( href.namespaceURI, 'http://www.w3.org/1999/xlink' );
+
+		component.teardown();
+	}
+};
diff --git a/test/generator/svg-xlink/main.html b/test/generator/svg-xlink/main.html
new file mode 100644
index 0000000000..d24b1510a7
--- /dev/null
+++ b/test/generator/svg-xlink/main.html
@@ -0,0 +1,7 @@
+<svg>
+	<defs>
+		<circle id='stamp' r='10' fill='blue'/>
+	</defs>
+
+	<use xlink:href='#stamp' x='20' y='20'/>
+</svg>
diff --git a/test/helpers.js b/test/helpers.js
index d1f0539a6c..d55923d93f 100644
--- a/test/helpers.js
+++ b/test/helpers.js
@@ -65,6 +65,10 @@ function cleanChildren ( node ) {
 		}
 
 		if ( child.nodeType === 3 ) {
+			if ( node.namespaceURI === 'http://www.w3.org/2000/svg' && node.tagName !== 'text' && node.tagName !== 'tspan' ) {
+				node.removeChild( child );
+			}
+
 			child.data = child.data.replace( /\s{2,}/, '\n' );
 
 			// text

From 8eb540db6d39162deacd75d3c3fcf3a881a796b0 Mon Sep 17 00:00:00 2001
From: Rich Harris <richard.a.harris@gmail.com>
Date: Tue, 24 Jan 2017 17:32:13 -0500
Subject: [PATCH 4/4] -> v1.6.6

---
 CHANGELOG.md | 5 +++++
 package.json | 2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 0d0733ef18..503fe6466c 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,5 +1,10 @@
 # Svelte changelog
 
+## 1.6.6
+
+* Omit text from comment anchors ([#247](https://github.com/sveltejs/svelte/issues/247))
+* Handle `xlink` attributes ([#264](https://github.com/sveltejs/svelte/issues/264))
+
 ## 1.6.5
 
 * Handle `<!doctype>` declarations ([#255](https://github.com/sveltejs/svelte/pull/255))
diff --git a/package.json b/package.json
index 40c191f029..e361ad1ad6 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "svelte",
-  "version": "1.6.5",
+  "version": "1.6.6",
   "description": "The magical disappearing UI framework",
   "main": "compiler/svelte.js",
   "files": [