non-bug-fix-patch. Removed the need for using a GHashTable in svg/svg-color.cpp

[Samuel Chase]

Removed the need for using a GHashTable in svg/svg-color.cpp.
I used a map<string, unsigned long> instead.

[Samuel Chase (samebchase)]

Inkscape segfaults with this patch.

[Samuel Chase (samebchase)]

Made some more changes.

This seems to work.

[Samuel Chase (samebchase)]

Removes some typcasting.

[Samuel Chase (samebchase)]

Even more prettified.

[Samuel Chase (samebchase)]

fixed bug in previous patch

[Kris (kris-degussem)]

Thanks for the patch.

Patch committed in trunk revision 11152 along with the following modifications:
- check on colors size changed in empty check (should be better according to the code checker cppcheck: " Checking for 'colors' emptiness might be inefficient. Using colors.empty() instead of colors.size() can be faster. colors.size() can take linear time but colors.empty() is guaranteed to take constant time.")
- I additionally converted some existing c-type variable casts to their c++ counterparts