chiark / gitweb /
where-vessels: No notes is default. Better treatment of lack of notes/ownership
[ypp-sc-tools.main.git] / yarrg / where-vessels
index f9f6879..b926492 100755 (executable)
@@ -101,8 +101,11 @@ proc nextarg {} {
     return $v
 }
 
-set notes_loc vessel-notes
+set notes_loc {}
 set scraper {./yppedia-ocean-scraper --chart}
+set info_cache _vessel-info-cache
+set info_source rsync.yarrg.chiark.net::yarrg/vessel-info
+set no_owners 0
 
 proc parseargs {} {
     global ai argv
@@ -118,6 +121,7 @@ proc parseargs {} {
            --clipboard-file { load-clipboard-file [nextarg] }
            --local-html-dir { lappend scraper --local-html-dir=[nextarg] }
            --notes { glset notes_loc [nextarg] }
+           --vessel-info-source { glset info_source [nextarg] }
            --debug { incr debug }
            default { badusage "unknown option $arg" }
        }
@@ -128,12 +132,19 @@ proc parseargs {} {
 
 proc argdefaults {} {
     global ocean notes_loc pirate scraper
-    if {![info exists ocean] || ![info exists pirate]} {
+    if {![info exists ocean] ||
+        (![info exists pirate] && [string length $notes_loc])} {
        set cmd {./yarrg --find-window-only --quiet}
        if {[info exists ocean]} { lappend cmd --ocean $ocean }
        if {[info exists pirate]} { lappend cmd --pirate $pirate }
        manyset [split [eval exec $cmd] " "] ocean pirate
+       if {![llength $ocean] || ![llength $pirate]} {
+           error "$ocean $pirate ?"
+       }
     }
+    if {![info exists pirate]} { set pirate {}; glset no_owners 1 }
+    if {![string length $notes_loc]} { glset no_owners 1 }
+
     lappend scraper $ocean
 }
 
@@ -243,23 +254,35 @@ proc display-note-infos {} {
 
 #---------- vessel properties ----------
 
-proc vesselclasses-init {} {
-    global vc_game2code vc_code2abbrev vc_code2full vc_codes
-    set vcl {
-       smsloop         am      sl      Sloop
-       lgsloop         bm      ct      Cutter
-       dhow            cm      dh      Dhow
-       longship        dm      ls      Longship
-       baghlah         em      bg      Baghlah
-       merchbrig       fm      mb      {Merchant Brig}
-       warbrig         gm      wb      {War Brig}
-       xebec           hm      xe      Xebec
-       merchgal        jm      mg      {Merchant Galleon}
-       warfrig         im      wf      {War Frigate}
-       grandfrig       km      gf      {Grand Frigate}
+proc info-cache-update {} {
+    global info_source info_cache
+    file mkdir $info_cache
+    exec sh -c "cp -u icons/* $info_cache/."
+
+    if {[string length $info_source]} {
+       set cmdl [list \
+                 rsync -udLKtOzm \
+                 --exclude=*~ --exclude=*.bak --exclude=.* --exclude=*.tmp \
+                 $info_source/ $info_cache 2>@ stderr]
+       debug "INFO-CACHE $cmdl"
+       eval exec $cmdl
     }
+
+    set f [open $info_cache/vessel-info]
+    glset vessel_size_data [read $f]
+    close $f
+}
+
+proc vesselinfo-init {} {
+    global vc_game2code vc_code2abbrev vc_code2full vc_codes
+
+    global vessel_size_data
+    manyset $vessel_size_data sizeinfos subclassinfos
+
     set vc_codes {}
-    foreach {game code abbrev full} $vcl {
+    foreach {game code abbrev full} $sizeinfos {
+       if {![regexp {^[a-z][a-z]$} $code code]} { error "bad code" }
+       if {![regexp {^[a-z][a-z]$} $abbrev abbrev]} { error "bad abbrev" }
        lappend vc_codes $code
        set vc_game2code($game) $code
        set vc_code2abbrev($code) $abbrev
@@ -267,23 +290,37 @@ proc vesselclasses-init {} {
        load-icon $abbrev
     }
 
+    global vsc_code2report
+    global vsc_game2code
+    set vsc_game2code(null) {}
+    set vsc_code2report() Ordinary
+    set vsc_code2report(!) "(Special/L.E.)"
+    foreach {game code full} $subclassinfos {
+       if {![regexp {^[A-Z]$} $code code]} { error "bad code" }
+       set vsc_game2code($game) $code
+       set vsc_code2report($code) $full
+    }
+
     load-icon atsea
+    set owners {ours dot query}
+    foreach b $owners { load-icon $b }
     foreach a {battle borrow dot} {
-       foreach b {ours dot query} {
-           load-icon-combine $a $b
-       }
+       load-icon $a
+       foreach b $owners { load-icon-combine $a $b }
     }
 }
 
 proc load-icon {icon} {
-    image create bitmap icon/$icon -file icons/$icon.xbm
+    global info_cache
+    image create bitmap icon/$icon -file $info_cache/$icon.xbm
 }
 
 proc load-icon-combine {args} {
+    global info_cache
     set cmd {}
     set delim "pnmcat -lr "
     foreach icon $args {
-       append cmd $delim " <(xbmtopbm icons/$icon.xbm)"
+       append cmd $delim " <(xbmtopbm $info_cache/$icon.xbm)"
        set delim " <(pbmmake -white 1 1)"
     }
     append cmd " | pbmtoxbm"
@@ -293,11 +330,11 @@ proc load-icon-combine {args} {
 
 proc code-lockown2icon {lockown} {
     manyset [split $lockown ""] lock notown
-    return icon/[
-                lindex {battle borrow dot} $lock
-               ]+[
-                  lindex {ours dot query} $notown
-                 ]
+    set l "
+        [lindex {battle borrow dot} $lock]
+         [lindex {ours dot query {} {} dot} $notown]
+    "
+    if {[llength $l]} { return icon/[join $l +] } { return {} }
 }
 
 proc canvas-horiz-stack {xvar xoff y bind type args} {
@@ -305,6 +342,7 @@ proc canvas-horiz-stack {xvar xoff y bind type args} {
     upvar 1 canvas canvas
     set id [eval $canvas create $type [expr {$x+$xoff}] $y $args]
     set bbox [$canvas bbox $id]
+#   debug "CANVAS-HORIZ-STACK $type $x $xoff $id $bbox [list $args]"
     set x [lindex $bbox 2]
     $canvas bind $id <ButtonPress> $bind
     return $id
@@ -314,7 +352,7 @@ proc code2canvas {code canvas x yvar qty qtylen bind} {
     global vc_code2abbrev
     upvar 1 $yvar y
 
-    manyset [split $code _] inport class subclass lockown xabbrev
+    manyset [split $code _] inport size subclass lockown xabbrev
 
     set stackx $x
     incr stackx 2
@@ -333,8 +371,14 @@ proc code2canvas {code canvas x yvar qty qtylen bind} {
        incr stackx
     }
     
+    upvar #0 vc_code2abbrev($size) vcabb
+    if {![info exists vcabb]} {
+       set vcabb vc-$size
+       image create bitmap icon/$vcabb -data \
+           [exec pbmtext -builtin fixed $size | pnminvert | pnmcrop >t.pnm]
+    }
     canvas-horiz-stack stackx -1 $imy $bind \
-           image -anchor nw -image icon/$vc_code2abbrev($class)
+           image -anchor nw -image icon/$vcabb
 
     if {[string length $subclass]} {
        canvas-horiz-stack stackx 0 $y $bind \
@@ -343,9 +387,12 @@ proc code2canvas {code canvas x yvar qty qtylen bind} {
     }
 
     incr stackx
-    canvas-horiz-stack stackx 0 $imy $bind \
-       image -anchor nw -image [code-lockown2icon $lockown]
-    incr stackx
+    set lockownicon [code-lockown2icon $lockown]
+    if {[string length $lockownicon]} {
+       canvas-horiz-stack stackx 0 $imy $bind \
+           image -anchor nw -image $lockownicon
+       incr stackx
+    }
     
     if {[string length $xabbrev]} {
        canvas-horiz-stack stackx 0 $y $bind \
@@ -368,36 +415,257 @@ proc code2canvas {code canvas x yvar qty qtylen bind} {
 proc show-report-decode {code} {
     global vc_code2full
 
-    manyset [split $code _] inport classcode subclass lockown xabbrev
+    smash-prepare
+
+    manyset [split $code _] inport sizecode subclass lockown xabbrev
     manyset [split $lockown ""] lock notown
     
     report-set inport [lindex {{At Sea} {In port}} $inport]
-    report-set class $vc_code2full($classcode)
 
-    switch -exact $subclass {
-       {} { report-set subclass {Ordinary} }
-       F { report-set subclass {"Frost class"} }
-       default { report-set subclass "Subclass \"$subclass\"" }
+    upvar #0 vc_code2full($sizecode) sizefull
+    upvar #0 smash_sizeinexact($sizecode) sizeinexact
+    set size_report $sizefull
+    if {[info exists sizeinexact]} { set size_report "($sizefull+)" }
+    report-set size $size_report
+
+    global smash_subclass
+    if {$smash_subclass >= 2} {
+       report-set subclass "(Any class)"
+    } elseif {[
+              upvar #0 vsc_code2report($subclass) subclass_report
+              info exists subclass_report
+             ]} {
+       report-set subclass $subclass_report
+    } else {
+       report-set subclass "Class \"$subclass\""
     }
 
     report-set lock [lindex {
        {Battle ready} {Unlocked} {Locked}
+       {(All lock states)} {(Not battle ready)}
     } $lock]
 
-    switch -exact $notown {
-       0 { report-set own "Yours" }
-       1 { report-set own "Other pirate's" }
-       2 { report-set own "Owner unknown" }
-       default { report-set own "?? $notown" }
+    global no_owners notes_loc
+    if {!$no_owners} {
+       switch -exact $notown {
+           0 { report-set own "Yours" }
+           1 { report-set own "Other pirate's" }
+           2 { report-set own "Owner unknown" }
+           3 { report-set own "(All ownerships)" }
+           4 - 5 { report-set own "(Yours/unknown)" }
+           default { report-set own "?? $notown" }
+       }
     }
 
-    if {[string length $xabbrev]} {
-       report-set xabbrev "Notes flags: $xabbrev"
+    global smash_xabbrev_map
+    if {![string length $notes_loc]} {
+    } elseif {[llength $smash_xabbrev_map]} {
+       if {[string length $xabbrev]} {
+           report-set xabbrev "(Flags: $xabbrev)"
+       } else {
+           report-set xabbrev "(No flags)"
+       }
     } else {
-       report-set xabbrev "No flags in notes"
+       if {[string length $xabbrev]} {
+           report-set xabbrev "Notes flags: $xabbrev"
+       } else {
+           report-set xabbrev "No flags in notes"
+       }
     }
 }
 
+#---------- common to smashing and filtering ----------
+
+proc make-control {parent ctrl label ekind} {
+    debug "MAKE-CONTROL [list $parent $ctrl $label $ekind]"
+    label $parent.lab_$ctrl -text $label -justify left
+    $ekind $parent.$ctrl
+    manyset [grid size $parent] dummy row
+    incr row
+    grid configure $parent.lab_$ctrl -row $row -column 0 -sticky nw -pady 4
+    grid configure $parent.$ctrl -row $row -column 1 -sticky w -pady 3
+    return $parent.$ctrl
+}
+
+proc begin-control-grid {cw count rows inrow} {
+    if {!$inrow} { set inrow [expr {($count + $rows) / $rows}] }
+    upvar #0 control_grid_properties($cw) props
+    set props [list $rows $inrow]
+    return $cw
+}
+
+proc make-control-grid-elem {cw kind ix ekind args} {
+    upvar #0 control_grid_properties($cw) props
+    manyset $props rows inrow
+
+    set ew $cw.$ix
+
+    debug "MAKE-CONTROL-GRID-ELEM $cw $kind $ix $ekind $rows $inrow $ew"
+
+    eval [list $ekind $ew] $args
+
+    switch -exact $kind {
+       ix {
+           grid configure $ew -sticky sw \
+               -row [expr {$ix / $inrow}] \
+               -column [expr {$ix % $inrow}]
+       }
+       final {
+           grid configure $ew -sticky se \
+               -row [expr {$rows-1}] \
+               -column [expr {$inrow-1}]
+       }
+       default {
+           error "$kind ?"
+       }
+    }
+    return $ew
+}
+
+proc control-tickbox-flip {varsvn values} {
+    upvar #0 $varsvn vars
+    foreach val $values {
+       set vars($val) [expr {!$vars($val)}]
+    }
+    redraw-needed c.-tickbox-flip $varsvn $values
+}
+
+proc populate-control-grid-tickboxes {cw rows inrow varsvn values flipvalues
+                                   label_kind valvn default_get label_get} {
+    debug "POPULATE-CONTROL-GRID-TICKBOXES $cw $rows $inrow $varsvn\
+             [list $values] $label_kind $valvn"
+
+    upvar #0 $varsvn vars
+    upvar 1 $valvn val
+    set count [llength $values]
+
+    begin-control-grid $cw $count $rows $inrow
+
+    for {set ix 0} {$ix < $count} {incr ix} {
+       set val [lindex $values $ix]
+       set vars($val) [uplevel 1 $default_get]
+       set ew [make-control-grid-elem $cw ix $ix checkbutton \
+                   -variable ${varsvn}($val) \
+                   -font fixed \
+                   -command [list redraw-needed c.-g.-tickbox $cw $val]]
+       $ew configure -$label_kind [uplevel 1 $label_get]
+       switch -exact $label_kind {
+           image { $ew configure -height 16 }
+       }
+    }
+    [make-control-grid-elem $cw final invert button] \
+       configure \
+       -text flip -command [list control-tickbox-flip $varsvn $flipvalues] \
+       -padx 0 -pady 0
+}
+
+#---------- smashing ----------
+
+proc smash-code {code} {
+    manyset [split $code _] inport size subclass lockown xabbrev
+
+    upvar #0 smash_sizemap($size) smsize
+
+    global smash_subclass
+    if {$smash_subclass > 1} {
+       set subclass {}
+    } elseif {$smash_subclass && [string length $subclass]} {
+       set subclass !
+    }
+
+    global smash_owner
+    switch $smash_owner {
+       0 { }
+       1 { regsub {[12]$} $lockown 5 lockown }
+       2 {
+           if {[regexp {^0.} $lockown]} {
+               # battle ready / all lock states
+               set lockown 03
+           } elseif {[regexp {^.0} $lockown]} {
+               # not battle ready / yours
+               set lockown 40
+           } else {
+               # state (not battle ready) / not known to be yours
+               regsub {.$} $lockown 4 lockown
+           }
+       }
+       3 { regsub {.$} $lockown {3} lockown }
+       4 { set lockown 33 }
+    }
+
+    global smash_xabbrev_map
+    set xabbrev [string map $smash_xabbrev_map $xabbrev]
+
+    return [join [list $inport $smsize $subclass $lockown $xabbrev] _]
+}
+
+proc smash-prepare {} {
+    global vc_codes smash_sizemap smash_size smash_sizeinexact
+    set mapto {}
+    catch { unset smash_sizeplus }
+    foreach size $vc_codes {
+       if {!$smash_size($size)} {
+           set mapto $size
+       } else {
+           set smash_sizeinexact($mapto) 1
+       }
+       set smash_sizemap($size) $mapto
+    }
+
+    global smash_xabbrev_a smash_xabbrev_b smash_xabbrev_map
+    set smash_xabbrev_map {}
+    foreach a [split $smash_xabbrev_a ""] b [split $smash_xabbrev_b ""] {
+       if {![string length $a]} continue
+       lappend smash_xabbrev_map $a $b
+    }
+    debug "SMASH-PREPAE xabbrev_map=[list $smash_xabbrev_map]"
+}
+
+proc make-smasher {sma label ekind} {
+    return [make-control .smash $sma $label $ekind]
+}
+
+proc make-radio-smasher {sma label variable descs rows inrow} {
+    set w [make-smasher $sma $label frame]
+    begin-control-grid $w [llength $descs] $rows $inrow
+    for {set i 0} {$i < [llength $descs]} {incr i} {
+       make-control-grid-elem $w ix $i \
+           radiobutton \
+           -variable $variable -value $i -command redraw-needed \
+           -text [lindex $descs $i]
+    }
+}
+
+proc make-smashers {} {
+    global vc_codes vc_code2abbrev
+    set cw [make-smasher size "Size\n round\n down" frame]
+    populate-control-grid-tickboxes $cw 2 0 smash_size \
+       $vc_codes [lrange $vc_codes 1 end] \
+       image val { expr 0 } { expr {"icon/$vc_code2abbrev($val)"} }
+    $cw.0 configure -state disabled
+
+    glset smash_subclass 0
+    make-radio-smasher subclass Class smash_subclass \
+       {Show Normal/LE Hide} 1 0
+
+    global no_owners
+    glset smash_owner [expr {$no_owners ? 3 : 0}]
+    make-radio-smasher owner Owner smash_owner \
+       {Show Yours? {For you} Lock Hide} 2 3
+
+    set cw [make-smasher xabbrev "Flags" frame]
+    foreach ix {1 3} ab {a b} width {14 12} {
+       set vn smash_xabbrev_$ab
+       global $vn
+       set $vn {}
+       entry $cw.$ix -textvariable $vn -width $width
+       trace add variable $vn write [list redraw-needed $vn]
+    }
+    set ix 0
+    foreach str {y/ / /d} { label $cw.$ix -text $str; incr ix 2 }
+    eval pack [lsort [winfo children $cw]] -side left
+}
+
 #---------- filtering ----------
 
 set filters {}
@@ -429,6 +697,7 @@ proc filter-default/lockown {lockown} {
 proc filter-says-yes/lockown {codel} {
     set lockown [lindex $codel 3]
     upvar #0 filter_lockown($lockown) yes
+    debug "FILTER-SAYS-YES/LOCKOWN $codel $lockown $yes"
     return $yes
 }
 
@@ -448,43 +717,22 @@ proc filter-says-yes/xabbre {codel} {
     return [regexp -- $filter_xabbre $xabbrev]
 }
 
-proc filter-tickbox-flip {fil} {
-    upvar #0 filter_$fil vars
-    set values [filter-values/$fil]
-    foreach val $values {
-       set vars($val) [expr {!$vars($val)}]
-    }
-    redraw-needed
-}
-
 proc make-tickbox-filter {fil label rows inrow} {
-    upvar #0 filter_$fil vars
-    set fw [make-filter tickbox $fil $label frame]
     set values [filter-values/$fil]
-    set nvalues [llength $values]
-    if {!$inrow} {
-       set inrow [expr {($nvalues + $rows) / $rows}]
-    }
-    set noicons [catch { info args filter-icon/$fil }]
-    for {set ix 0} {$ix < $nvalues} {incr ix} {
-       set val [lindex $values $ix]
-       set vars($val) [filter-default/$fil $val]
-       checkbutton $fw.$ix -variable filter_${fil}($val) \
-           -font fixed -command redraw-needed
-       if {!$noicons} {
-           $fw.$ix configure -image [filter-icon/$fil $val] -height 16
-       } else {
-           $fw.$ix configure -text [filter-map/$fil $val]
-       }
-       grid configure $fw.$ix -sticky sw \
-           -row [expr {$ix / $inrow}] \
-           -column [expr {$ix % $inrow}]
+
+    if {![catch { info args filter-icon/$fil }]} {
+       set label_get { filter-icon/$fil $val }
+       set label_kind image
+    } else {
+       set label_get { filter-map/$fil $val }
+       set label_kind text
     }
-    button $fw.invert -text flip -command [list filter-tickbox-flip $fil] \
-       -padx 0 -pady 0
-    grid configure $fw.invert -sticky se \
-       -row [expr {$rows-1}] \
-       -column [expr {$inrow-1}]
+
+    set fw [make-filter $fil $label frame]
+
+    populate-control-grid-tickboxes $fw $rows $inrow filter_$fil \
+       $values $values \
+       $label_kind val { filter-default/$fil $val } $label_get
 }
 
 proc entry-filter-changed {fw fil n1 n2 op} {
@@ -511,7 +759,7 @@ proc make-entry-filter {fil label def} {
     global filterentered_$fil
     upvar #0 filter_$fil realvar
     set realvar $def
-    set fw [make-filter entry $fil $label frame]
+    set fw [make-filter $fil $label frame]
     entry $fw.entry -textvariable filterentered_$fil
     label $fw.error
     glset def_background [$fw.error cget -background]
@@ -520,15 +768,10 @@ proc make-entry-filter {fil label def} {
     pack $fw.entry $fw.error -side top -anchor w
 }
 
-proc make-filter {kind fil label ekind} {
+proc make-filter {fil label ekind} {
     global filters
-    label .filter.lab_$fil -text $label -justify left
-    $ekind .filter.$fil
     lappend filters $fil
-    set nfilters [llength $filters]
-    grid configure .filter.lab_$fil -row $nfilters -column 0 -sticky nw -pady 4
-    grid configure .filter.$fil -row $nfilters -column 1 -sticky w -pady 3
-    return .filter.$fil
+    return [make-control .filter $fil $label $ekind]
 }
 
 proc make-filters {} {
@@ -545,7 +788,6 @@ proc filterstyle-changed {n1 n2 op} {
 
 proc filters-say-yes {code} {
     global filters filterstyle
-    debug "filters-say-yes $code"
     set codel [split $code _]
     set lockown [lindex $codel 3]
     switch -exact $filterstyle {
@@ -557,8 +799,12 @@ proc filters-say-yes {code} {
     }
     
     foreach fil $filters {
-       if {![filter-says-yes/$fil $codel]} { return 0 }
+       if {![filter-says-yes/$fil $codel]} {
+           debug "FILTERS-SAY-YES $code NO $fil"
+           return 0
+       }
     }
+    debug "FILTERS-SAY-YES $code YES $filters"
     return 1
 }
     
@@ -571,16 +817,28 @@ proc vessel {vin} {
     set codel {}
     lappend codel [errexpect-arrayget-boolean vi inPort]
 
-    set gameclass [errexpect-arrayget vi vesselClass]
-    upvar #0 vc_game2code($gameclass) class
-    if {![info exists class]} { errexpect-error "unexpected vesselClass"}
-    lappend codel $class
+    set gamesize [errexpect-arrayget vi vesselClass]
+    upvar #0 vc_game2code($gamesize) size
+    if {![info exists size]} {
+       set size "($gamesize)"
+       upvar #0 vc_code2abbrev($size) vcabb
+       set vcabb vc-$size
+       set data [exec pbmtext -builtin fixed " $gamesize " \
+                | pnminvert | pnmcrop | pbmtoxbm]
+       debug "INVENTED ICON $vcabb $data"
+       image create bitmap icon/$vcabb -data $data
+           
+       global vc_code2full
+       set vc_code2full($size) "Type \"$gamesize\""
+    }
+    lappend codel $size
 
-    set subclass [errexpect-arrayget vi vesselSubclass]
-    switch -exact $subclass {
-       null            { lappend codel {} }
-       icy             { lappend codel F }
-       default         { lappend codel ($subclass) }
+    set gamesubclass [errexpect-arrayget vi vesselSubclass]
+    upvar #0 vsc_game2code($gamesubclass) subclass
+    if {[info exists subclass]} {
+       lappend codel $subclass
+    } else {
+       lappend codel ($gamesubclass)
     }
 
     switch -exact [errexpect-arrayget vi isLocked]/[ \
@@ -625,7 +883,7 @@ proc vessel {vin} {
     lappend newnotes [list $vid $realname $owner $xabbrev]
     set kk "$island [join $codel _]"
     upvar #0 found($kk) k
-    lappend k [list $vid $realname]
+    lappend k [list $vid $realname $owner]
  
     debug "CODED $kk $vid $realname"
 }
@@ -695,7 +953,7 @@ proc load-chart {} {
        yppedia_chart_parse(\*STDIN, (new IO::File ">/dev/null"),
                sub { sprintf "%d %d", @_; },
                sub { printf "archlabel %d %d %s\n", @_; },
-               sub { printf "island %s %s\n", @_; },
+               sub { printf "island %s {%s} %s\n", @_; },
                sub { printf "league %s %s %s.\n", @_; },
                sub { printf STDERR "warning: %s: incomprehensible: %s", @_; }
                        );
@@ -703,8 +961,18 @@ proc load-chart {} {
     }]]
 }
 
-
-set scale 16
+proc init-scales {} {
+    global scales scaleix scale
+    set defscale 16
+    set scales {1 2 3 4 5 6 8}
+    set e12 {10 12 15 18 22 27 33 39 47 56 68 82}
+    foreach t $e12 {
+       if {$t < $defscale} { set scaleix [llength $scales] }
+       lappend scales $t
+    }
+    foreach t [lrange $e12 0 6] { lappend scales [expr {$t * 10}] }
+    set scale [lindex $scales $scaleix]
+}
 
 proc coord {c} {
        global scale
@@ -712,17 +980,19 @@ proc coord {c} {
 }
 
 proc chart-got/archlabel {args} { }
-proc chart-got/island {x y args} {
-#      debug "ISLE $x $y $args"
+proc chart-got/island {x y isle sizecol} {
+       debug "ISLE $x $y $isle $sizecol"
        global canvas isleloc
-       set isleloc($args) [list $x $y]
+       set isleloc($isle) [list $x $y]
        set sz 5
 #      $canvas create oval \
 #              [expr {[coord $x] - $sz}] [expr {[coord $y] - $sz}] \
 #              [expr {[coord $x] + $sz}] [expr {[coord $y] + $sz}] \
 #              -fill blue
+       set colour "#888"
+       if {[string match *_col $sizecol]} { set colour black }
        $canvas create text [coord $x] [coord $y] \
-               -text $args -anchor s
+               -text $isle -anchor s -fill $colour
 }
 proc chart-got/league {x1 y1 x2 y2 kind} {
 #      debug "LEAGUE $x1 $y1 $x2 $y2 $kind"
@@ -735,15 +1005,32 @@ proc chart-got/league {x1 y1 x2 y2 kind} {
        }
 }
 
-proc redraw-needed {} {
+proc debug-filter-array {array} {
+    upvar #0 $array a
+    set m " FILTER $array"
+    foreach k [lsort [array names a]] {
+       append m " $k=$a($k)"
+    }
+    debug $m
+}
+
+proc redraw-needed {args} {
     global redraw_after
-    debug "REDRAW NEEDED"
+    debug "REDRAW NEEDED $args"
     if {[info exists redraw_after]} return
+
+    global filterstyle
+    debug " FILTER style $filterstyle"
+    debug-filter-array filter_size
+    debug-filter-array filter_lockown
+    global filter_xabbre
+    debug " FILTER xabbre $filter_xabbre"
+
     set redraw_after [after 250 draw]
 }
 
 proc draw {} {
-    global chart found isleloc canvas redraw_after islandnames
+    global chart found isleloc canvas redraw_after islandnames smfound
 
     catch { after cancel $redraw_after }
     catch { unset redraw_after }
@@ -756,15 +1043,27 @@ proc draw {} {
        eval chart-got/$proc [lrange $l 1 end]
     }
 
-    set islandnames {}
-    set lastislandname {}
+    smash-prepare
+
+    catch { unset smfound }
     foreach key [lsort [array names found]] {
-       set c [llength $found($key)]
-#      debug "SHOWING $key $c"
        regexp {^(.*) (\S+)$} $key dummy islandname code
 
        if {![filters-say-yes $code]} continue
 
+       set smcode [smash-code $code]
+       debug "smashed $code => $smcode"
+       set smkey "$islandname $smcode"
+       foreach vessel $found($key) { lappend smfound($smkey) $vessel }
+    }
+
+    set islandnames {}
+    set lastislandname {}
+    foreach smkey [lsort [array names smfound]] {
+       set c [llength $smfound($smkey)]
+       regexp {^(.*) (\S+)$} $smkey dummy islandname code
+       debug "SHOWING [list $smkey $c $islandname $code l=$lastislandname]"
+
        if {[string compare $lastislandname $islandname]} {
                manyset $isleloc($islandname) x y
                set x [coord $x]
@@ -805,6 +1104,7 @@ proc parser-control-create {w base invokebuttontext etl_title} {
     toplevel $eb
     wm withdraw $eb
     wm title $eb "where-vessels - $etl_title"
+    wm protocol $eb WM_DELETE_WINDOW [list wm withdraw $eb]
 
     label $eb.title -text $etl_title
     pack $eb.title -side top
@@ -813,17 +1113,17 @@ proc parser-control-create {w base invokebuttontext etl_title} {
     pack $eb.close -side bottom
 
     frame $eb.emsg -bd 2 -relief groove
-    label $eb.emsg.lab -text "Error:"
+    label $eb.emsg.lab -anchor nw -text "Error:"
     text $eb.emsg.text -height 1
-    pack $eb.emsg.text -side bottom
+    pack $eb.emsg.text -side bottom -fill x
     pack $eb.emsg.lab -side left
 
-    pack $eb.emsg -side top -pady 2
+    pack $eb.emsg -side top -pady 2 -fill x
 
     frame $eb.text -bd 2 -relief groove
-    pack $eb.text -side bottom -pady 2
+    pack $eb.text -side bottom -pady 2 -fill both -expand y
     
-    label $eb.text.lab
+    label $eb.text.lab -anchor nw
 
     text $eb.text.text -width 85 \
        -xscrollcommand [list $eb.text.xscroll set] \
@@ -836,10 +1136,15 @@ proc parser-control-create {w base invokebuttontext etl_title} {
     scrollbar $eb.text.yscroll -orient vertical \
        -command [list $eb.text.text yview]
 
-    grid configure $eb.text.lab -row 0 -column 0 -sticky w
-    grid configure $eb.text.text -row 1 -column 0
+    grid configure $eb.text.lab -row 0 -column 0 -sticky w -columnspan 2
+    grid configure $eb.text.text -row 1 -column 0 -sticky news
     grid configure $eb.text.yscroll -sticky ns -row 1 -column 1
     grid configure $eb.text.xscroll -sticky ew -row 2 -column 0
+    grid rowconfigure $eb.text 0 -weight 0
+    grid rowconfigure $eb.text 1 -weight 1
+    grid rowconfigure $eb.text 2 -weight 0
+    grid columnconfigure $eb.text 0 -weight 1
+    grid columnconfigure $eb.text 1 -weight 0
 }
 
 proc parser-control-ok-core {w base background show} {
@@ -950,10 +1255,14 @@ proc islandnames-handler {offset maxchars} {
 #---------- main user interface ----------
 
 proc widgets-setup {} {
-    global canvas debug pirate ocean filterstyle
+    global canvas debug pirate ocean filterstyle notes_loc
 
-    wm geometry . 1024x600
-    wm title . "where-vessels - $pirate on the $ocean ocean"
+    wm geometry . 1200x800
+    if {[string length $pirate]} {
+       wm title . "where-vessels - $pirate on the $ocean ocean"
+    } else {
+       wm title . "where-vessels - $ocean ocean"
+    }
 
     #----- map -----
 
@@ -966,9 +1275,13 @@ proc widgets-setup {} {
     #----- control panels and filter -----
 
     frame .cp
+    frame .smash -relief groove -bd 2 -padx 1
     frame .filter -relief groove -bd 2 -padx 1
     frame .islands -pady 2
-    pack .cp .filter .islands -side top
+    pack .cp .filter .islands .smash -side top
+
+    label .smash.title -text {Display/combine details}
+    grid .smash.title -row 0 -column 0 -columnspan 2
 
     set filterstyle 1
     trace add variable filterstyle write filterstyle-changed
@@ -999,9 +1312,11 @@ proc widgets-setup {} {
     frame .cp.ctrl.zoom
     pack .cp.ctrl.zoom -side top
 
-    button .cp.ctrl.zoom.out -text - -font {Courier 16} -command {zoom /2} -pady 0
-    button .cp.ctrl.zoom.in  -text + -font {Courier 16} -command {zoom *2} -pady 0
-    pack .cp.ctrl.zoom.out .cp.ctrl.zoom.in -side left
+    foreach inout {out in} minplus {- +} {
+       button .cp.ctrl.zoom.$inout -text $minplus -font {Courier 16} \
+           -command "zoom ${minplus}1" -pady 0
+       pack .cp.ctrl.zoom.$inout -side left
+    }
 
     parser-control-create .cp.ctrl.acquire \
        acquire Acquire \
@@ -1012,9 +1327,13 @@ proc widgets-setup {} {
     parser-control-create .cp.ctrl.notes \
        notes "Reload notes" \
        "Vessel notes loading report" \
-       
+
     pack .cp.ctrl.notes -side top -pady 2
 
+    if {![string length $notes_loc]} {
+       .cp.ctrl.notes.do configure -state disabled
+    }  
+       
     #----- island name count and copy -----
 
     label .islands.count
@@ -1047,7 +1366,7 @@ proc widgets-setup {} {
     #pack .cp.report.code -side top
     pack configure .cp.report.details -fill x
 
-    foreach sw {inport class subclass lock own xabbrev} {
+    foreach sw {inport size subclass lock own xabbrev} {
        label .cp.report.details.$sw -text { }
        pack .cp.report.details.$sw -side top -anchor w
     }
@@ -1056,6 +1375,8 @@ proc widgets-setup {} {
 proc report-set {sw val} { .cp.report.details.$sw configure -text $val }
 
 proc show-report {islandname code} {
+    global no_owners
+    
     .cp.report.island configure -text $islandname
 
     .cp.report.abbrev delete all
@@ -1068,22 +1389,38 @@ proc show-report {islandname code} {
     show-report-decode $code
 
     set kk "$islandname $code"
-    upvar #0 found($kk) k
+    upvar #0 smfound($kk) k
 
     .cp.report.list delete 0 end
 
-    foreach entry $k {
-       manyset $entry vid name
-       .cp.report.list insert end $name
+    foreach elem $k {
+       manyset $elem vid name owner
+       lappend owned($owner) $name
+    }
+
+    foreach owner [lsort [array names owned]] {
+       if {[string length $owner]} {
+           set owndesc "$owner's"
+       } else {
+           set owndesc "Owner unknown"
+       }
+       if {!$no_owners} {
+           .cp.report.list insert end "$owndesc:"
+       }
+       foreach name $owned($owner) {
+           .cp.report.list insert end " $name"
+       }
     }
 }
 
-proc zoom {extail} {
-    global scale canvas
-    set nscale [expr "\$scale $extail"]
-    debug "ZOOM $scale $nscale"
-    if {$nscale < 1 || $nscale > 200} return
-    set scale $nscale
+proc zoom {amt} {
+    global scaleix scales scale canvas
+    incr scaleix $amt
+    if {$scaleix < 0} { set scaleix 0 }
+    set nscales [llength $scales]
+    if {$scaleix >= $nscales} { set scaleix [expr {$nscales-1}] }
+    set scale [lindex $scales $scaleix]
+    debug "ZOOM $amt $scaleix $scale"
     draw
 }
 
@@ -1132,19 +1469,39 @@ proc invoke_notes {} {
 
 #---------- main program ----------
 
+init-scales
 parseargs
-vesselclasses-init
 argdefaults
 httpclientsetup where-vessels
+info-cache-update
+vesselinfo-init
 load-chart
 widgets-setup
 make-filters
+make-smashers
 
 set notes_data {}
 if {[catch { parse-clipboard } emsg]} {
     puts stderr "$emsg\n$errorInfo"
     exit 1
 }
-after idle invoke_notes
+if {[string length $notes_loc]} {
+    after idle invoke_notes
+}
 
 draw
+
+if {$debug} {
+    package require Tclx
+    commandloop -async \
+       -prompt1 { return "where-vessels% " } \
+       -prompt2 { return "> " }
+}
+
+# some runes I use:
+#
+# offline development
+#   ./where-vessels --notes ~/vessel-notes --vessel-info-source '' --pirate Aristarchus --ocean Midnight --debug --local-html-dir . --clipboard-file ~/clipboard-aristarchus
+#
+# updating published vessel info
+#   rsync -r --exclude=\*~ yarrg/icons/. ijackson@chiark.greenend.org.uk:/home/ftp/users/ijackson/yarrg/vessel-info/.