diff --git a/lib/jsduck/doc/subproperties.rb b/lib/jsduck/doc/subproperties.rb
index a71c1ae81cda0c3ede3471d62c6a37288a2fd4fd..d63a936b90e80622aeaa65318bdc0b7872e9f8cc 100644
--- a/lib/jsduck/doc/subproperties.rb
+++ b/lib/jsduck/doc/subproperties.rb
@@ -1,4 +1,5 @@
 require 'jsduck/util/singleton'
+require 'jsduck/logger'
 
 module JsDuck
   module Doc
@@ -22,13 +23,12 @@ module JsDuck
       #         {:name => "baz"}]},
       #     {:name => "zap"},
       #
-      # NOTE: It returns two values: the resulting nested items array
-      # and warnings array of :subproperty warnings that should get
-      # printed out.
-      def nest(raw_items)
+      # Secondly it takes a position argument which is used for
+      # logging warnings when bogus subproperty syntax is encountered.
+      def nest(raw_items, pos)
         # First item can't be namespaced, if it is ignore the rest.
         if raw_items[0] && raw_items[0][:name] =~ /\./
-          return [raw_items[0]], []
+          return [raw_items[0]]
         end
 
         # build name-index of all items
@@ -39,7 +39,6 @@ module JsDuck
         # Otherwise look up the parent of item and add it as the
         # property of that parent.
         items = []
-        warnings = []
         raw_items.each do |it|
           if it[:name] =~ /^(.+)\.([^.]+)$/
             it[:name] = $2
@@ -48,14 +47,15 @@ module JsDuck
               parent[:properties] = [] unless parent[:properties]
               parent[:properties] << it
             else
-              warnings << "Ignoring subproperty #{$1}.#{$2}, no parent found with name '#{$1}'."
+              msg = "Ignoring subproperty #{$1}.#{$2}, no parent found with name '#{$1}'."
+              Logger.warn(:subproperty, msg, pos[:filename], pos[:linenr])
             end
           else
             items << it
           end
         end
 
-        return items, warnings
+        return items
       end
 
     end
diff --git a/lib/jsduck/tag/cfg.rb b/lib/jsduck/tag/cfg.rb
index ba158ee6d62d1f0fe778ae75cc2064a51f842aec..addcefa6e420018b799c6159f605dca994b4a0b2 100644
--- a/lib/jsduck/tag/cfg.rb
+++ b/lib/jsduck/tag/cfg.rb
@@ -26,14 +26,8 @@ module JsDuck::Tag
       h[:name] = p[:name]
       h[:type] = p[:type]
       h[:default] = p[:default]
-      h[:properties] = nest_properties(tags, pos)
+      h[:properties] = JsDuck::Doc::Subproperties.nest(tags, pos)[0][:properties]
       h[:required] = true if p[:optional] == false
     end
-
-    def nest_properties(tags, pos)
-      items, warnings = JsDuck::Doc::Subproperties.nest(tags)
-      warnings.each {|msg| JsDuck::Logger.warn(:subproperty, msg, pos[:filename], pos[:linenr]) }
-      items[0][:properties]
-    end
   end
 end
diff --git a/lib/jsduck/tag/param.rb b/lib/jsduck/tag/param.rb
index 5013946854055ca5df41475887f058284c4d7c9f..6cc07946f68616212105fc54658ec2c5729883d6 100644
--- a/lib/jsduck/tag/param.rb
+++ b/lib/jsduck/tag/param.rb
@@ -21,9 +21,7 @@ module JsDuck::Tag
     end
 
     def process_doc(h, tags, pos)
-      items, warnings = JsDuck::Doc::Subproperties.nest(tags)
-      warnings.each {|msg| JsDuck::Logger.warn(:subproperty, msg, pos[:filename], pos[:linenr]) }
-      h[:params] = items
+      h[:params] = JsDuck::Doc::Subproperties.nest(tags, pos)
     end
   end
 end
diff --git a/lib/jsduck/tag/property.rb b/lib/jsduck/tag/property.rb
index 8f289c4eea333949e81731757be1a0796752a7de..37dbf39a8aa8714947d98a257bcd87fb4f60953c 100644
--- a/lib/jsduck/tag/property.rb
+++ b/lib/jsduck/tag/property.rb
@@ -22,13 +22,7 @@ module JsDuck::Tag
       # Type might also come from @type, don't overwrite it with nil.
       h[:type] = p[:type] if p[:type]
       h[:default] = p[:default]
-      h[:properties] = nest_properties(tags, pos)
-    end
-
-    def nest_properties(tags, pos)
-      items, warnings = JsDuck::Doc::Subproperties.nest(tags)
-      warnings.each {|msg| JsDuck::Logger.warn(:subproperty, msg, pos[:filename], pos[:linenr]) }
-      items[0][:properties]
+      h[:properties] = JsDuck::Doc::Subproperties.nest(tags, pos)[0][:properties]
     end
   end
 end
diff --git a/lib/jsduck/tag/return.rb b/lib/jsduck/tag/return.rb
index f86666ca96ccb8897f8400f11ed0f5227df561ef..6f64547fd8a5a0215648e2aff1be4557d92505f2 100644
--- a/lib/jsduck/tag/return.rb
+++ b/lib/jsduck/tag/return.rb
@@ -30,14 +30,8 @@ module JsDuck::Tag
         :type => ret[:type] || "Object",
         :name => ret[:name] || "return",
         :doc => ret[:doc] || "",
-        :properties => nest_properties(tags, pos)[0][:properties]
+        :properties => JsDuck::Doc::Subproperties.nest(tags, pos)[0][:properties],
       }
     end
-
-    def nest_properties(tags, pos)
-      items, warnings = JsDuck::Doc::Subproperties.nest(tags)
-      warnings.each {|msg| JsDuck::Logger.warn(:subproperty, msg, pos[:filename], pos[:linenr]) }
-      items
-    end
   end
 end