typekit.rb: minor fix for regex

the mistake shouldn't harm. but the fix won't hurt either.

Signed-off-by: Martin Zenzes <martin.zenzes@dfki.de>
typekit.rb: fix comments

- "included_files" _is_ an array, see typekit.rb:934
- the parameter _was_ removed/renamed in 2013

Signed-off-by: Martin Zenzes <martin.zenzes@dfki.de>
typekit.rb: more explicit syntax

not sure if this is good or bad -- am not a ruby guy. but writing "Hash.new" explicitly is more consistent with the rest of the code.

Signed-off-by: Martin Zenzes <martin.zenzes@dfki.de>
typekit.rb: add warning for non-existing line in "source_file_line"

general muttering: these kind of checks... i don't get it... if the ":"
is missing in the "source_file_line", the API-agreement is broken by
someone. which is a crystal-clear error! orogen should bail out.  I
don't know about the quality concerning the "source_file_line" feature
when backed by gccxml though...

Signed-off-by: Martin Zenzes <martin.zenzes@dfki.de>
typekit.rb: include_for_type: add special handling for '/std/string' and '/string'

otherwise the function would return ["vector", "boost/cstdint.hpp"] as
it sees the template-argument of the Container

for uint8_t-like types: return "boost/cstdint.hpp". why not use
"stdint.h"?

Signed-off-by: Martin Zenzes <martin.zenzes@dfki.de>
templates: remove various unconditional includes of headers

"vector", "string" and "boost/cstdint.h" -- is not needed as long as the
rest of the code does its job?

Signed-off-by: Martin Zenzes <martin.zenzes@dfki.de>
Revert "always add the system directories to the list of include dirs"

This reverts commit d8f39106e7fde8f1574ed753b7feb79a5ea1cddf.

Conflicts:
	lib/orogen/gen/typekit.rb
typekit.rb: fix missing typekit-name in "orogen_include"

this commit grew larger than anticipated, due to the rather convoluted
design of orogen...

as mentioned here: https://github.com/orocos-toolchain/orogen/issues/7#issuecomment-52231833

fixed to the best of my knowledge ;-) including the typekit-name in the
added line removes the doubled mention of "ReadOnlyPointer.hpp"

also renamed the "name" variable to a more precise "ptr_name".
reformulated the comments and renamed one of the "inc" variables to
"or_inc", just for a little bit extra-clarity.

Signed-off-by: Martin Zenzes <martin.zenzes@dfki.de>