Andreas Raab a écrit :
The *one* tradeoff I wasn't willing to make for the sake of my users was
to break their code; was to make their code non-loadable for absolutely
no benefit for these users. Cleaning up two methods and fifty lines of
code isn't worth breaking six years of work from
Hi guys
I would like to build a set of canonical code formatting convention for Pharo.
I need your help. Now take time before replying :)
I would like to structure the discussion and proceed step by step. So at max I
would like to discuss one or two formatting approach per mail.
Once we agree I
If you feel that this is important we can take some time and port the code from
VW.
Let us know I can ask cyrille to give a try.
Stef
On Feb 27, 2010, at 10:56 PM, Nicolas Cellier wrote:
2010/2/27 Levente Uzonyi le...@elte.hu:
On Sat, 27 Feb 2010, Richard Durr wrote:
So what speaks
I would use a formatter, like this you can make it really consistent.
The RBConfigurableFormatter is pretty good in that, the only thing I
don't like is the placing of the square brackets. I should look into
fixing that, otherwise it formats exactly the way I would format
myself (and in the way
Lukas do you have some code samples.
Something that I particularly hate is the following
self bla ifTrue: [
...
] ifFalse: [
...
]
That is see in the code.
I want
self bla
ifTrue: [ ...
2010/2/28 Igor Stasenko siguc...@gmail.com:
Hi, i'm also did some hacking. I uploaded XTream-Wrappers-sig.1 into
SqS/XTream.
There is a basic XtreamWrapper class, which should work transparently
for any stream (hopefully ;).
Next, in subclass i created converter. Sure thing i could also add
On 28 February 2010 12:00, Nicolas Cellier
nicolas.cellier.aka.n...@gmail.com wrote:
2010/2/28 Igor Stasenko siguc...@gmail.com:
Hi, i'm also did some hacking. I uploaded XTream-Wrappers-sig.1 into
SqS/XTream.
There is a basic XtreamWrapper class, which should work transparently
for any
11238
-
- Issue 2089: Case insensitive search for MethodFinder/MessageNames tx marcus
- Issue 2097: SecureHashAlgorithm raising an error upon empty input (from
squeak)
- Issue 2087: Compacting all methodDictionaries (from squeak)
- Cycles cutting tx Jannik
Stef (I will be off playing
On 28 February 2010 11:58, Stéphane Ducasse stephane.duca...@inria.fr wrote:
Lukas do you have some code samples.
Something that I particularly hate is the following
self bla ifTrue: [
...
] ifFalse: [
...
]
That is see in the code.
I
http://video.google.com/videoplay?docid=-4216011961522818645#
The subtitle is misleading
How Open Source Projects Survive Poisonous People (And You Can
Too)
I think that we can learn from them :)
The video is much more that about Poisonous people. A lot of interested advices
Looks good to me
If one day we get a cooler one we could get
self class isVariable anotherObject class isVariable
ifTrue:
[ 1 to: (self basicSize min: anotherObject basicSize)
do: [ :ind |
self basicAt: ind put: (anotherObject basicAt: ind) ] ]
But, if there is only single block, i usually typing:
self bla ifTrue: [
code.
code.
]
Yes I see
I would prefer
self bla ifTrue: [
code.
code. ]
for me I try
self bla ifTrue: [ code ].
or
self bla
ifTrue: [ code.
code.]
but I'm not happy because
2010/2/28 Igor Stasenko siguc...@gmail.com:
On 28 February 2010 12:00, Nicolas Cellier
nicolas.cellier.aka.n...@gmail.com wrote:
2010/2/28 Igor Stasenko siguc...@gmail.com:
Hi, i'm also did some hacking. I uploaded XTream-Wrappers-sig.1 into
SqS/XTream.
There is a basic XtreamWrapper
11239
-
- Issue 2026: RecentMessageSet fixes. Tx Pavel :)
- Issue 2066: Conversions from/to hex representations in ByteArray (from
squeak). Now we can read and print ByteArray in hex with upper and lower case.
Stef
well a final one (yes noury we arrive now).
On 28 February 2010 12:45, Nicolas Cellier
nicolas.cellier.aka.n...@gmail.com wrote:
2010/2/28 Igor Stasenko siguc...@gmail.com:
On 28 February 2010 12:00, Nicolas Cellier
nicolas.cellier.aka.n...@gmail.com wrote:
2010/2/28 Igor Stasenko siguc...@gmail.com:
Hi, i'm also did some hacking. I
On 28 February 2010 12:44, Stéphane Ducasse stephane.duca...@inria.fr wrote:
But, if there is only single block, i usually typing:
self bla ifTrue: [
code.
code.
]
Yes I see
I would prefer
self bla ifTrue: [
code.
code. ]
for me I try
self bla ifTrue: [ code ].
yes,
Johan and Romain are fine too. This morning a new shake woke me up...
It should be like this for the next weeks.
Alexandre
On 28 Feb 2010, at 06:44, Stéphane Ducasse wrote:
Let us know. Seems to be a really big earthquake.
Probably at santiago some old houses suffered but not the new ones.
So will anybody be at the CeBIT?
RD
___
Pharo-project mailing list
Pharo-project@lists.gforge.inria.fr
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
I would remove cleanUpMethods and move what is worth and working to
cleanUpForRelease.
Cheers,
Adrian
On Feb 25, 2010, at 13:49 , Mariano Martinez Peck wrote:
Try to evaluate ScriptLoader new cleanUpMethods
and you will see that in ScriptLoader cleanUpMethods
you do:
Delay
It worked for me the first time I tried, but not the second time (the VM
eventually crashed).
Already in Squeak 3.9 it was sometimes impossible to break endless recursions.
So I'm not sure if this related to changes in Pharo 1.0.
Adrian
On Feb 26, 2010, at 16:50 , Lukas Renggli wrote:
Is
2010/2/28 Igor Stasenko siguc...@gmail.com:
On 28 February 2010 12:45, Nicolas Cellier
nicolas.cellier.aka.n...@gmail.com wrote:
2010/2/28 Igor Stasenko siguc...@gmail.com:
On 28 February 2010 12:00, Nicolas Cellier
nicolas.cellier.aka.n...@gmail.com wrote:
2010/2/28 Igor Stasenko
Good to now that you are going fine ;-) I'm currently in Paris. I'm
leaving to Ha Noi this afternoon but there is a big tempest in Europe
right now, a lot of flights have been canceled ...
On Sun, Feb 28, 2010 at 12:56 PM, Alexandre Bergel alexan...@bergel.eu wrote:
Johan and Romain are fine
I wasn't implying that this was the fault of Pharo. I just noticed it
suddenly and I somehow thought that this worked at some point in the
past, but I might be wrong.
Lukas
On 28 February 2010 13:29, Adrian Lienhard a...@netstyle.ch wrote:
It worked for me the first time I tried, but not the
Sure, no worries. In any case it would be nice if it always worked...
Adrian
On Feb 28, 2010, at 13:52 , Lukas Renggli wrote:
I wasn't implying that this was the fault of Pharo. I just noticed it
suddenly and I somehow thought that this worked at some point in the
past, but I might be
On 28 February 2010 14:33, Nicolas Cellier
nicolas.cellier.aka.n...@gmail.com wrote:
2010/2/28 Igor Stasenko siguc...@gmail.com:
On 28 February 2010 12:45, Nicolas Cellier
..
Anyway, main contributor of UTF8Decoder inefficiency is
Unicodevalue: and leadingChar handling:
btw, why its
2010/2/28 Igor Stasenko siguc...@gmail.com:
On 28 February 2010 14:33, Nicolas Cellier
nicolas.cellier.aka.n...@gmail.com wrote:
2010/2/28 Igor Stasenko siguc...@gmail.com:
On 28 February 2010 12:45, Nicolas Cellier
..
Anyway, main contributor of UTF8Decoder inefficiency is
On 28 February 2010 15:44, Nicolas Cellier
nicolas.cellier.aka.n...@gmail.com wrote:
2010/2/28 Igor Stasenko siguc...@gmail.com:
On 28 February 2010 14:33, Nicolas Cellier
nicolas.cellier.aka.n...@gmail.com wrote:
2010/2/28 Igor Stasenko siguc...@gmail.com:
On 28 February 2010 12:45, Nicolas
Stef,
It pains me to have to be a pain about this. I never use a space after a
colon; what other people do is their business, but I much prefer having the
space gone. Other formatting concerns are far less trivial to me than a
personal preference.
Lukas is correct about using a formatter,
Hi all,
I'm preparing a presentation and just wondering if anyone out there
is doing bioinformatics with Smalltalk?
Cheers,
Hernán
___
Pharo-project mailing list
Pharo-project@lists.gforge.inria.fr
On Feb 28, 2010, at 1:58 AM, Stéphane Ducasse wrote:
Lukas do you have some code samples.
Something that I particularly hate is the following
self bla ifTrue: [
...
] ifFalse: [
...
]
I'd like to understand what it is about the above you
On Sun, 28 Feb 2010, James Foster wrote:
On Feb 28, 2010, at 1:58 AM, Stéphane Ducasse wrote:
Lukas do you have some code samples.
Something that I particularly hate is the following
self bla ifTrue: [
...
] ifFalse: [
...
]
I'd
I sent an email last week, and saw no answer, so far. So I've put up
what I have.
Stéphane Ducasse wrote:
Yanni excellent to have it on squeaksource.
Now did you contact the maintainer (if any) or the people that developed it
(I like it a lot) to mention that?
Because at least they should
Adrian,
Horrible is an understatement :) Your version is a big improvement. I
frequently put ending brackets on their own line with indentation to aid in
matching, but not always.
I do not (much) care what conventions the benevolent dictators select for
Pharo, but I **do** care that it not
we should open a ticket
On Sun, Feb 28, 2010 at 2:15 PM, Adrian Lienhard a...@netstyle.ch wrote:
Sure, no worries. In any case it would be nice if it always worked...
Adrian
On Feb 28, 2010, at 13:52 , Lukas Renggli wrote:
I wasn't implying that this was the fault of Pharo. I just
I wouldn't be so sure.
http://html5demos.com/
http://jqtouch.com/
On Feb 27, 2010, at 4:41 PM, Schwab,Wilhelm K wrote:
there is no way in hell that HTML+CSS+JavaScript is going to keep up
___
Pharo-project mailing list
Hi Bill,
Don't worry. Nobody wants to force you to use his conventions :)
The question really only concerns the code of PharoCore. I think a consistent
way of code formatting really is a good idea, even if not everybody agrees with
each rule.
Cheers,
Adrian
On Feb 28, 2010, at 18:43 ,
I would like to add to this discussion.
The issue on breaking code compatibility /per se/ it is not the central
issue, the real trouble is to have to expend too much time to learn how to
re-compatibilize the cote, or use [the] new construct that is not deprecated.
I'm not an active Python
Adrian,
I appreciate the reassurance, and agree that conventions are a good idea. I
further agree with Lukas that a formatter is a good way to get there, but want
to make the case for doing that in a way that does not force the hands of
end-user (which I appear to have done).
Bill
There is the further concern about the browser. It wastes screen space (which
is at a premium) and adds weight and distracting functionality. There is
something to be said for the a fat client app that does what it needs to do and
nothing else.
-Original Message-
From:
http://gforge.inria.fr/frs/download.php/26559/PharoCore-1.1-11239-UNSTABLE.zip
--
Marcus Denker -- http://www.marcusdenker.de
INRIA Lille -- Nord Europe. Team RMoD.
___
Pharo-project mailing list
Pharo-project@lists.gforge.inria.fr
thanks adrian
On Feb 28, 2010, at 2:11 PM, Adrian Lienhard wrote:
I've attached the coding conventions that we use at Cmsbox and netstyle.ch.
They closely follow the suggestions of Kent Beck's Smalltalk Best Practice
Patterns.
According to these rules I would format the following
+ 10
I'd like to understand what it is about the above you hate.
It's like C code, not like a message send. Do you write code like this?
aCollection do: [ :each |
...
] separatedBy: [
...
]
That is see in the code.
I want
self bla
ifTrue: [ ...
you know go (the language) includes in the language the formatting so there is
no discussion.
Now I really like to see smalltalk code and not C code because.
It slows me down a lot when I read and I read a lot of code. :)
Stef
On Feb 28, 2010, at 8:51 PM, Adrian Lienhard wrote:
Hi
Hi James
Lukas do you have some code samples.
Something that I particularly hate is the following
self bla ifTrue: [
...
] ifFalse: [
...
]
I'd like to understand what it is about the above you hate.
See levente
In addition I have to apy
I've commented a version of RBConfigurableFormatter that can format
#copyFrom: to the following code. This comes pretty close to the
document of Adrian:
ObjectcopyFrom: anotherObject
Copy to myself all instance variables I have in common with
anotherObject. This is dangerous because it
Thanks lukas
I've commented
commited :) I guess I saw in on the rss feed :)
a version of RBConfigurableFormatter that can format
#copyFrom: to the following code. This comes pretty close to the
document of Adrian:
ObjectcopyFrom: anotherObject
Copy to myself all instance
I've commented
commited :) I guess I saw in on the rss feed :)
Yeah.
can we get rid of the
:ind |
Select 'ind', in the context menu select 'refactor source', 'rename
temporary'. Then give the new name 'index', press 'ok'. Done.
Lukas
--
Lukas Renggli
http://www.lukas-renggli.ch
On Feb 28, 2010, at 22:12 , Stéphane Ducasse wrote:
---
1
to: (mine size min: his size)
do: [ :index |
(mine at: index) = (his at: index) ifTrue: [
self
instVarAt: index
put:
The configurable formatter can be told from how many keyword messages
on it should put them on multiple lines. Also you can give exceptions
of messages that you always/never want to put on multiple lines.
Lukas
On 28 February 2010 22:29, Adrian Lienhard a...@netstyle.ch wrote:
On Feb 28, 2010,
On Feb 28, 2010, at 10:28 PM, Lukas Renggli wrote:
I've commented
commited :) I guess I saw in on the rss feed :)
Yeah.
can we get rid of the
:ind |
Select 'ind', in the context menu select 'refactor source', 'rename
temporary'. Then give the new name 'index', press 'ok'. Done.
On 28 February 2010 22:38, Stéphane Ducasse stephane.duca...@inria.fr wrote:
On Feb 28, 2010, at 10:28 PM, Lukas Renggli wrote:
I've commented
commited :) I guess I saw in on the rss feed :)
Yeah.
can we get rid of the
:ind |
Select 'ind', in the context menu select 'refactor
lol
I was really skeptical :)
:ind | at the beginning of the line.
Ahh, that was the mail program that broke it there.
___
Pharo-project mailing list
Pharo-project@lists.gforge.inria.fr
Stef,
I have never liked forced formatting; good scanners and parsers have allowed us
to move beyond that. Again, I have no real concern over how you chose to
format the code in the base system; I would care greatly about any attempt to
build that into the language (since you bring it up) or
53 matches
Mail list logo