[Freeciv-Dev] (PR#2292) Patch: Artillery attack against sea units

["Guest" <rt-guest@xxxxxxxxxxx>]

<URL: http://bugs.freeciv.org/Ticket/Display.html?id=2292 >

It seems that we have to tweak the handle_unit_move_request() function 
in the server/unithand.c file in new Freeciv versions. 

There is one thing I have not understood. The handle_unit_move_request
() function checks whether an attacking unit is a bombarder. If so, it 
also checks that the attacked tile is not ocean. If we remove this 
check, then we let all the bombarders attack sea units. However, 
according to the units.ruleset file, the Bombarder flag is not 
attached to any unit. Is it goining to be used someday? If so, is it a 
good idea to combine bombarder functionality with the sea attack 
functionality or should we introduce a new flag for these purposes? 

Sincerely,
Alexander Sayenko