STARK: Remove / update TODOs

engines/stark/resources/image.cpp

@@ -106,7 +106,6 @@ void Image::printData() {
 }
 
 int Image::indexForPoint(const Common::Point &point) const {
-	// TODO: This doesn't necessarily get the innermost polygon
 	int index = -1;
 	for (uint32 i = 0; i < _polygons.size(); i++) {
 		if (isPointInPolygon(_polygons[i], point)) {

engines/stark/resources/item.cpp

@@ -295,7 +295,6 @@ Visual *ItemVisual::getVisual() {
 }
 
 int ItemVisual::getHotspotIndexForPoint(const Common::Point &point) {
-	// TODO: This breaks rather weirdly on subtype 6 and 10
 	Anim *anim = getAnim();
 	if (anim) {
 		return anim->getPointHotspotIndex(point);

engines/stark/services/dialogplayer.cpp

@@ -104,8 +104,6 @@ void DialogPlayer::removeLastOnlyOption() {
 void DialogPlayer::buildOptions() {
 	Resources::Dialog::TopicArray availableTopics = _currentDialog->listAvailableTopics();
 
-	// TODO: This is very minimal, complete
-
 	for (uint i = 0; i < availableTopics.size(); i++) {
 		Option option;
 

engines/stark/services/resourceprovider.cpp

@@ -123,8 +123,8 @@ Resources::Location *ResourceProvider::getLocation(uint16 level, uint16 location
 	return nullptr;
 }
 
-// TODO: This approach doesn't carry over all the stuff that it should, perhaps we should avoid reloading?
 void ResourceProvider::pushAndChangeLocation(int16 level, int16 location) {
+	// TODO: Keep track of the inventory state
 	pushCurrentLocation();
 	requestLocationChange(level, location);
 }

engines/stark/ui/fmvplayer.cpp

@@ -49,7 +49,6 @@ FMVPlayer::~FMVPlayer() {
 }
 
 void FMVPlayer::play(const Common::String &name) {
-	// TODO: Clear existing
 	Common::SeekableReadStream *stream = StarkArchiveLoader->getExternalFile(name, "Global/");
 	if (!stream) {
 		warning("Could not open %s", name.c_str());