[Seaside-dev] Re: [Seaside Commits] Seaside 3.0: Seaside-Tools-Web-dkh.54.mcz

[Dale Henrichs]

~~ nil is optimized in GemStone, so it is a habit to use that ... I'll change it

Dale

----- "Julian Fitzell" <jfitzell at gmail.com> wrote:

| Our usual pattern for this would be "isNil ifFalse:"...
| 
| Julian
| 
| On Fri, Nov 13, 2009 at 11:15 AM,
| <squeak-dev-noreply at lists.squeakfoundation.org> wrote:
| > Dale Henrichs uploaded a new version of Seaside-Tools-Web to project
| Seaside 3.0:
| > http://www.squeaksource.com/Seaside30/Seaside-Tools-Web-dkh.54.mcz
| >
| > ==================== Summary ====================
| >
| > Name: Seaside-Tools-Web-dkh.54
| > Author: dkh
| > Time: 13 November 2009, 11:15:41 am
| > UUID: 79335fa6-65cd-480b-ade0-728823c053cd
| > Ancestors: Seaside-Tools-Web-jf.53
| >
| > - eliminate use of #ifNotNil:
| >
| > =============== Diff against Seaside-Tools-Web-jf.53
| ===============
| >
| > Item was changed:
| >  ----- Method: WAFileAttributeEditor>>renderConfigurationOn: (in
| category 'rendering') -----
| >  renderConfigurationOn: html
| >        | selectedLib |
| >        selectedLib := WAValueHolder new.
| >        html form: [
| >                html table: [
| >                 html tableRow: [
| >                        html tableData: [
| >                                        html select
| >                                                list: self
| libraries;
| >                                                callback: [:value |
| selectedLib contents: value ];
| >                                                size: 10 ].
| >                        html tableData: [
| >                                html anchor
| >                                                title: 'Configure
| selected file library';
| >                                                class: 'large';
| >                                                onClick: 'return
| submitForm(this)';
| > +                                               callback: [
| selectedLib contents ~~ nil ifTrue: [self configureLibrary:
| selectedLib contents default ] ];
| > -                                               callback: [
| selectedLib contents ifNotNil: [self configureLibrary: selectedLib
| contents default ] ];
| >                                                with: 'Configure'] ]
| ] ]!
| >
| >
| > _______________________________________________
| > commits mailing list
| > To unsubscribe, email commits-leave at lists.seaside.st
| > http://lists.seaside.st/listinfo/commits
| >
| _______________________________________________
| seaside-dev mailing list
| seaside-dev at lists.squeakfoundation.org
| http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev