Addressed IWYU complaints

Author: PatrikLundell

src/avatar.cpp

@@ -347,14 +347,15 @@ void avatar::set_active_mission( mission &cur_mission )
 
 void avatar::set_active_point_of_interest( point_of_interest active_point_of_interest )
 {
-    for( auto iter = points_of_interest.begin(); iter < points_of_interest.end(); iter++ ) {
+    for( auto iter : points_of_interest ) {
         // It's really sufficient to only check the position as used...
-        if( iter._Ptr->pos == active_point_of_interest.pos &&
-            iter._Ptr->text == active_point_of_interest.text ) {
+        if( iter.pos == active_point_of_interest.pos &&
+            iter.text == active_point_of_interest.text ) {
             this->active_point_of_interest = active_point_of_interest;
             active_mission = nullptr;
             return;
         }
+
     }
 
     debugmsg( "active point of interest %s is not in the points_of_interest list",
@@ -446,7 +447,7 @@ void avatar::add_point_of_interest( point_of_interest new_point_of_interest )
 void avatar::delete_point_of_interest( tripoint_abs_omt pos )
 {
     for( auto iter = points_of_interest.begin(); iter != points_of_interest.end(); iter++ ) {
-        if( iter._Ptr->pos == pos ) {
+        if( iter->pos == pos ) {
             points_of_interest.erase( iter );
 
             if( active_point_of_interest.pos == pos ) {

src/avatar.h

@@ -25,6 +25,7 @@
 #include "magic_teleporter_list.h"
 #include "mdarray.h"
 #include "memory_fast.h"
+#include "point.h"
 #include "type_id.h"
 #include "units.h"
 

src/mission_ui.cpp

@@ -18,6 +18,7 @@
 #include "npc.h"
 #include "faction.h"
 #include "output.h"
+#include "point.h"
 #include "string_formatter.h"
 #include "talker.h"
 #include "translation.h"
@@ -239,8 +240,8 @@ void mission_ui_impl::draw_controls()
         get_avatar().delete_point_of_interest( upoints_of_interest[selected_mission].pos );
     }
 
-    if( selected_tab != mission_ui_tab_enum::POINTS_OF_INTEREST && umissions.empty() ||
-        selected_tab == mission_ui_tab_enum::POINTS_OF_INTEREST && upoints_of_interest.empty() ) {
+    if( ( selected_tab != mission_ui_tab_enum::POINTS_OF_INTEREST && umissions.empty() ) ||
+        ( selected_tab == mission_ui_tab_enum::POINTS_OF_INTEREST && upoints_of_interest.empty() ) ) {
         static const std::map< mission_ui_tab_enum, std::string > nope = {
             { mission_ui_tab_enum::ACTIVE, translate_marker( "You have no active missions!" ) },
             { mission_ui_tab_enum::COMPLETED, translate_marker( "You haven't completed any missions!" ) },