[Gzz-commits] gzz ./TODO doc/pegboard/1017/PEG_1017.rst

[Asko Soukka]

CVSROOT:        /cvsroot/gzz
Module name:    gzz
Changes by:     Asko Soukka <address@hidden>    02/10/16 02:15:12

Modified files:
        .              : TODO 
        doc/pegboard/1017: PEG_1017.rst 

Log message:
        PEG1017

CVSWeb URLs:
http://savannah.gnu.org/cgi-bin/viewcvs/gzz/gzz/TODO.diff?tr1=1.295&tr2=1.296&r1=text&r2=text
http://savannah.gnu.org/cgi-bin/viewcvs/gzz/gzz/doc/pegboard/1017/PEG_1017.rst.diff?tr1=1.2&tr2=1.3&r1=text&r2=text

Patches:
Index: gzz/TODO
diff -c gzz/TODO:1.295 gzz/TODO:1.296
*** gzz/TODO:1.295      Tue Oct 15 08:48:24 2002
--- gzz/TODO    Wed Oct 16 02:15:12 2002
***************
*** 65,72 ****
        - porting: make sure all demos etc. work with plain mesa
      humppake:
        - port Mind* views from 0.6
-          - BFRaster probably insufficient 
-              (at least when its depth is hard coded ;)
        - culling of coordsys: coordsys may, at setparams time,
          decide not to be shown.
            - generalizes culledpaperquad
--- 65,70 ----
Index: gzz/doc/pegboard/1017/PEG_1017.rst
diff -c gzz/doc/pegboard/1017/PEG_1017.rst:1.2 
gzz/doc/pegboard/1017/PEG_1017.rst:1.3
*** gzz/doc/pegboard/1017/PEG_1017.rst:1.2      Tue Oct 15 17:23:18 2002
--- gzz/doc/pegboard/1017/PEG_1017.rst  Wed Oct 16 02:15:12 2002
***************
*** 3,21 ****
  =============================================================
  
  :Author:   Asko Soukka
! :Date:     $Date: 2002/10/15 21:23:18 $
! :Revision: $Revision: 1.2 $
  :Status:   Incomplete
  
! The reading depth in ``gzz.view.BFRaster`` is currently a public 
! attribute, which is only used within ``BFRaster.read()``. We could
! get rid of it by making it parameter for ``BFRaster.read()``.
! Since it no longer has default value, ``BFRaster``'s reading depth
! can't be negligently ignored by view designer.
!     
! Of course we can leave public depth attribute as it is and 
! make new ``BFRaster.read()`` simple to give it a new value and call
! the old ``read()``.
  
  Changes
  -------
--- 3,28 ----
  =============================================================
  
  :Author:   Asko Soukka
! :Date:     $Date: 2002/10/16 06:15:12 $
! :Revision: $Revision: 1.3 $
  :Status:   Incomplete
  
! The reading depth in ``gzz.view.BFRaster`` is currently a public
! attribute, which is used within ``BFRaster.read()``. Although, its
! publicity makes it readable from using context, publicity makes
! it also directly writable. Currently writing directly into the 
! attribute from the context is the only way to change raster's reading
! depth.
! 
! We could avoid this and use the depth attribute read only as the
! other attributes in raster's by making it a new parameter for 
! ``BFRaster.read()``.
! 
! Currently there is a default value for the depth attribute. If
! the new depth attribute is always obligatory the default value
! is never used. Should we add the new attribute by only overloading /
! not replacing the current ''BFRaster.read()'' and make still 
! possible to use the default value?
  
  Changes
  -------
***************
*** 24,37 ****
  
           gzz/view/BFRaster.java
  
!     -    /** The depth to which read() will create the raster.
!     -     */
!     -    public int depth = 5;
! 
!     -    public void read(Cell center, Dim[] dims) { 
!     +    public void read(Cell center, Dim[] dims, int depth) { 
! 
! 
  
  
  
--- 31,43 ----
  
           gzz/view/BFRaster.java
  
!          /** The depth to which read() will create the raster.
!           */
!          public int depth = 5;
! 
!     -    public void read(Cell center, Dim[] dims) {
!     +    public void read(Cell center, Dim[] dims, int rdepth) { 
!     +            depth = rdepth;