Remove ancient handling of nodes w/o offsets

Author: wooorm

index.js

@@ -98,19 +98,18 @@ export function toNlcst(tree, file, Parser, options = {}) {
 function one(config, node) {
   /** @type {number} */
   var start
-  /** @type {number} */
-  var end
 
   if (!config.ignore.includes(node.type)) {
-    // To do: next major — nodes should have offsets, so
-    // `config.location.toOffset` should be superfluous.
-    start = config.location.toOffset(pointStart(node))
-    end = config.location.toOffset(pointEnd(node))
+    start = node.position.start.offset
 
     if (config.source.includes(node.type)) {
       return patch(
         config,
-        [config.parser.tokenizeSource(config.doc.slice(start, end))],
+        [
+          config.parser.tokenizeSource(
+            config.doc.slice(start, node.position.end.offset)
+          )
+        ],
         start
       )
     }

test/index.js

@@ -120,23 +120,6 @@ test('mdast-util-to-nlcst', function (t) {
     'should fail when not given positional information (#2)'
   )
 
-  t.test('should accept nodes without offsets', function (st) {
-    var node = toNlcst(
-      {
-        type: 'text',
-        value: 'foo',
-        position: {start: {line: 1, column: 1}, end: {line: 1, column: 4}}
-      },
-      vfile({contents: 'foo'}),
-      ParseLatin
-    )
-
-    st.equal(node.position.start.offset, 0, 'should set starting offset')
-    st.equal(node.position.end.offset, 3, 'should set ending offset')
-
-    st.end()
-  })
-
   t.end()
 })