- Pharo-dev - lists.pharo.org

Re: {Spam?} [Pharo-project] [Update] 10055 + new release
by Bill Schwab Aug. 15, 2008

Aug. 15, 2008

Ok, the profiler to the rescue. Instantiating some exception sub-classes should tell us whether there is anything to be gained from a factory of some sort. A quick look does not show much going on on instantiation, so it is probably fine to create the appropriate sub-instance and set any useful aspects, and to pass that to the ouch-block. Bill Wilhelm K. Schwab, Ph.D. University of Florida Department of Anesthesiology PO Box 100254 Gainesville, FL 32610-0254 Email: bschwab(a)anest.ufl.edu Tel: (352) 846-1285 FAX: (352) 392-7029 >>> m.rueger(a)acm.org 8/15/2008 3:48 PM >>> On Fri, Aug 15, 2008 at 8:51 PM, Bill Schwab <BSchwab(a)anest.ufl.edu> wrote: > I think the answer is probably to pass the *class* of the exception. > The default then simply sends the correct #signalWith: message with the > file name as an argument. Hmm, but the instance might hold additional information. Michael _______________________________________________ Pharo-project mailing list Pharo-project(a)lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

1 0

Re: {Spam?} [Pharo-project] [Update] 10055 + new release
by Bill Schwab Aug. 15, 2008

Aug. 15, 2008

I think the answer is probably to pass the *class* of the exception. The default then simply sends the correct #signalWith: message with the file name as an argument. Bill Wilhelm K. Schwab, Ph.D. University of Florida Department of Anesthesiology PO Box 100254 Gainesville, FL 32610-0254 Email: bschwab(a)anest.ufl.edu Tel: (352) 846-1285 FAX: (352) 392-7029 >>> m.rueger(a)acm.org 08/15/08 11:39 AM >>> Alexandre Bergel wrote: > At the first glance readOnlyFileNamed:ifNone: seems to be a useful > primitive. But this having the file absent might be just one case. You > could have ifNotReadable:, ifParentFolderNotReadable:. > Also, an error when writing in a file may occur in many different > situation (disk full, file read only, ...). > You may also want to provide handler for each of these situation (e.g., > writeFileNamed: aName with: stream ifDiskFull: [...] > ifAlreadyExistingFile: [...]... Agree... >> I agree you are improving on the existing code, but I would prefer >> something like >> >> file := fd readOnlyFileNamed:name ifNone:[ ^self ]. a solution would be to pass in any Exception into the ifNone: block: file := fd readOnlyFileNamed:name ifNone:[:ex| ^self ]. Then depending on whether you are actually interested in the exact reason the file couldn't be created you could look at the exception. Michael _______________________________________________ Pharo-project mailing list Pharo-project(a)lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

2 1

[Pharo-project] Controller no view member anymore
by Hilaire Fernandes Aug. 15, 2008

Aug. 15, 2008

Hello, I saw that in recent change, I guess MVC clean up, the Controller class has lost it view instance variable. Are there any reason doing that change? For reference, http://en.wikipedia.org/wiki/Model-view-controller Hilaire -- http://blog.ofset.org/hilaire

1 0

[Pharo-project] Worth considering?
by Hilaire Fernandes Aug. 15, 2008

Aug. 15, 2008

To load Magma Beta version throught Monticello Configuration by Chris, I need the two included fixes in Pharo. Does it worth integrating these fixes in Pharo? Hilaire -- http://blog.ofset.org/hilaire

3 7

Re: {Spam?} [Pharo-project] [Update] 10055 + new release
by Bill Schwab Aug. 15, 2008

Aug. 15, 2008

Stef, I agree you are improving on the existing code, but I would prefer something like file := fd readOnlyFileNamed:name ifNone:[ ^self ]. This is analogous to #at:ifAbsent:, which does not set up an exception handler and raise and handle an exception for something that is known to be handled locally. Then, just as #at: _does_ raise an exception, there should be readOnlyFileNamed:aBlock ^self readOnlyFileNamed:name ifNone:[ FileDoesNotExist signalWith:name. ]. #signalWith: might not be the correct syntax (too tired to look<g>), but hopefully it makes the point that exceptions should be raised, but there should also be "long winded" methods that skip the overhead when it is possible to do so. Bill Wilhelm K. Schwab, Ph.D. University of Florida Department of Anesthesiology PO Box 100254 Gainesville, FL 32610-0254 Email: bschwab(a)anest.ufl.edu Tel: (352) 846-1285 FAX: (352) 392-7029 >>> stephane.ducasse(a)inria.fr 08/11/08 4:15 PM >>> ------------------------------------------------------ update10052 ISSUE #124 While browsing at referencer of FileDoesNotExistException, I discovered the following pattern that is used in many different places. I find this pretty ugly: -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-= file := [fd readOnlyFileNamed: keysFileName] on: FileDoesNotExistException do:[:ex| nil]. file ifNil:[^self]. "no keys file" -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-= The use of exception is completely outwitted. I would rather write: -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-= [file := fd readOnlyFileNamed: keysFileName] on: FileDoesNotExistException do: [:ex| ^ self]. -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=

3 2

[Pharo-project] Re: License agreement
by StÃ©phane Ducasse Aug. 15, 2008

Aug. 15, 2008

sure I think that mentioning that if the code is published in the pharo squeaksource repository and on any squeaksource repository it is MIT. (I have problem to see people using free resources for private project) would be a good addition. On Aug 15, 2008, at 4:17 AM, Bill Schwab wrote: > Stef, > > I need to sign and send you a copy of the agreement. The terms are > fine > - if I give something away, I intend to do just that, and MIT is fine > (anything not GPL is fine<g>). The only small concern I have is that > the agreement does not specify what distinguishes contributed code. I > gather from elsewhere that loading it into a repository is the > trigger. > Would there be any problem defining that in the agreement? I am > simply > looking ahead to conversations with lawyers; I need to at least talk > to > them about it - they get quite upset if left out of things. > > If you are agreeable to a change, it might be best to wait until we > see > what, if any, other concerns might arise before updating the web site. > > Bill > > > > > Wilhelm K. Schwab, Ph.D. > University of Florida > Department of Anesthesiology > PO Box 100254 > Gainesville, FL 32610-0254 > > Email: bschwab(a)anest.ufl.edu > Tel: (352) 846-1285 > FAX: (352) 392-7029 > >

2 1

[Pharo-project] From OBrowser cannot save pane code when in prettyprint mode
by Hilaire Fernandes Aug. 14, 2008

Aug. 14, 2008

Does one else has noted this bug? Hilaire -- http://blog.ofset.org/hilaire

1 0

Aug. 13, 2008

Begin forwarded message: > From: pharo-project-owner(a)lists.gforge.inria.fr > Date: August 12, 2008 10:00:04 PM CEDT > To: stephane.ducasse(a)free.fr > Subject: Issue 219 Object>>isTransparent > > You are not allowed to post to this mailing list, and your message has > been automatically rejected. If you think that your messages are > being rejected in error, contact the mailing list owner at > pharo-project-owner(a)lists.gforge.inria.fr. > > > From: stephane ducasse <stephane.ducasse(a)free.fr> > Date: August 12, 2008 10:00:02 PM CEDT > To: An open mailing list to discuss any topics related to an open- > source Smalltalk <Pharo-project(a)lists.gforge.inria.fr> > Subject: Issue 219 Object>>isTransparent > > > It would be nice that Object does not implement isTransparent. > Apparently when you put Beeper beep in Object>>isTransparent this > method in invoked. It would be good to check whether another > (graphical) object should not implement isTransparent > > From stef and marcus. > > > > >

1 0

Aug. 11, 2008

Hello everybody just an announce to let you know that we did a update. Still some code to harvest in the repository. One slice made by adrian about speed stream but we have mixed feeling about it with marcus. Stef and marcus PS: Now the new compiler can load in pharo update10050 This update includes a set of Mantis fixes (that also went into Squeak 3.10): http://code.google.com/p/pharo/issues/detail?id=27 http://code.google.com/p/pharo/issues/detail?id=31 http://code.google.com/p/pharo/issues/detail?id=32 http://code.google.com/p/pharo/issues/detail?id=30 mark testUnwindDebuggerWithStep as expected failure ------------------------------------------------------ update10051 Name: SLICE-remove-copyHtml-al.6 Author: al Time: 22 July 2008, 9:37:16 pm UUID: 9192a548-32f3-43ac-8477-236a754a2a93 Ancestors: SLICE-remove-copyHtml-dc.5 in addition to previous version by Damien - remove #command: senders and implementors - remove method references to deleted methods in MessageFinder - remove String>>asHtml, which in turn required to... - remove an etoys related mail message class (FancyMailComposition), which required to - remove some html code in EToys (tell a friend feature) ------------------------------------------------------ update10052 ISSUE #124 While browsing at referencer of FileDoesNotExistException, I discovered the following pattern that is used in many different places. I find this pretty ugly: -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-= file := [fd readOnlyFileNamed: keysFileName] on: FileDoesNotExistException do:[:ex| nil]. file ifNil:[^self]. "no keys file" -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-= The use of exception is completely outwitted. I would rather write: -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-= [file := fd readOnlyFileNamed: keysFileName] on: FileDoesNotExistException do: [:ex| ^ self]. -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-= ------------------------------------------------------ update10053 Faster printString for LargeInteger http://code.google.com/p/pharo/issues/detail?id=114 http://code.google.com/p/pharo/issues/detail?id=74 There exist 2 related issues that report endless loops when using a bad base. http://bugs.squeak.org/view.php?id=6710 http://bugs.squeak.org/view.php?id=7107 ------------------------------------------------------ update10054 - Monticello enhancements (slice + comments) - UndefinedObject>>from3DS: removed - TransformationMorph>>heading: - Integer>>printStringBase: - - -> := in PasteUpMorph - Issue 118: 0006846: MenuMorphs can drop mouse focus. Patch provided by rootbeer. ------------------------------------------------------ update10055 Small fixes so that the NewCompiler can load and recompile the complete system

1 0

Aug. 11, 2008

Hi alex I introduced your comments.... copying button, but we only get the 8 or 10 characters of the comments. It would be cool if we could get the complete comment copied and pasted. Stef

2 1