Re: [Rails-devel] [Rails-commits] 18xx/rails/game TileI.java, 1.11, 1.12 Tile.java, 1.20, 1.21

[brett l. <wak...@gm...>]

I know that, historically, we've commented out many changes. Let's
dispose of that practice.

If it's your intention to change behavior and not just temporarily
disable something then just go ahead and remove the methods.

---Brett.

On Sat, Nov 15, 2008 at 5:41 AM, Erik Vos <ev...@us...> wrote:
> Update of /cvsroot/rails/18xx/rails/game
> In directory 23jxhf1.ch3.sourceforge.com:/tmp/cvs-serv7515/rails/game
>
> Modified Files:
>        TileI.java Tile.java
> Log Message:
> Removed isLayableNow()
>
> Index: TileI.java
> ===================================================================
> RCS file: /cvsroot/rails/18xx/rails/game/TileI.java,v
> retrieving revision 1.11
> retrieving revision 1.12
> diff -C2 -d -r1.11 -r1.12
> *** TileI.java  4 Jun 2008 19:00:32 -0000       1.11
> --- TileI.java  15 Nov 2008 13:41:11 -0000      1.12
> ***************
> *** 39,43 ****
>      public boolean isUpgradeable();
>
> !     public boolean isLayableNow();
>
>      public List<TileI> getUpgrades(MapHex hex);
> --- 39,43 ----
>      public boolean isUpgradeable();
>
> !     //public boolean isLayableNow();
>
>      public List<TileI> getUpgrades(MapHex hex);
>
> Index: Tile.java
> ===================================================================
> RCS file: /cvsroot/rails/18xx/rails/game/Tile.java,v
> retrieving revision 1.20
> retrieving revision 1.21
> diff -C2 -d -r1.20 -r1.21
> *** Tile.java   30 Jun 2008 20:35:29 -0000      1.20
> --- Tile.java   15 Nov 2008 13:41:11 -0000      1.21
> ***************
> *** 328,334 ****
>       * @return
>       */
> !     public boolean isLayableNow() {
> !         return GameManager.getInstance().getCurrentPhase().isTileColourAllowed(colourName);
> !     }
>
>      /**
> --- 328,334 ----
>       * @return
>       */
> !     // public boolean isLayableNow() {
> !     //    return GameManager.getInstance().getCurrentPhase().isTileColourAllowed(colourName);
> !     //}
>
>      /**
>
>