(Maint) Fix format of method definitions

Throughout the puppet codebase we use the method definition format that
puts parens around the params of the method.
(Maint) Remove unneeded setting of origin

The origin that is set on the model is not used when transforming to an
AST. Instead the AstTransformer is created with the filename that it is
transforming and uses that for the file of all of the AST elements.
(Maint) Remove is_adaptable?

The Adapter code for tracking extra information on the Pops models had
extra functionality that was never used: is_adaptable?. This allowed
objects to opt-out of being adaptable. However, there is no code using
this, yet it adds extra responds_to? checks for every call to add
information onto models.
(Maint) Simplify tracking of file origins

The Puppet::Pops::API::Origin system for tracking where a parsed model
came from was in anticipation of possible future expansions. Those
expansions are not around and so the extra complexity is simply adding
extra maintenance. This removes that complexity and should make the
origin tracking more straightforward.