You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@deltaspike.apache.org by Mark Struberg <st...@yahoo.de> on 2011/12/28 10:25:38 UTC
//X TODO
Hi!
Whenever you spot the following sequence in a class:
//X TODO some comment about the 'why'
then it means that there is some unfinished stuff lying around...
I'd say we now start adding stuff and mark all parts which cause sentiments with such a comment and clean this mess up in a 2nd step later.
LieGrue,
strub
Re: //X TODO
Posted by Matthias Wessendorf <ma...@apache.org>.
On Wed, Jan 4, 2012 at 4:20 PM, Mark Struberg <st...@yahoo.de> wrote:
> IF they are not used excessively.
+1
that was they point ;-)
-M
> Often this is a review marker for myself or others...
> Of course they should not remain in the code for longer than necessary.
>
> I usually let maven generate a taglist which I work through regularly.
>
> LieGrue,
> strub
>
>
> ----- Original Message -----
>> From: Matthias Wessendorf <ma...@apache.org>
>> To: deltaspike-dev@incubator.apache.org
>> Cc:
>> Sent: Wednesday, January 4, 2012 4:09 PM
>> Subject: Re: //X TODO
>>
>> FWIW
>>
>> http://twitter.com/#!/abailly/status/154506876594290689
>>
>> -M
>>
>> On Tue, Jan 3, 2012 at 10:02 PM, Jason Porter <li...@gmail.com>
>> wrote:
>>> +1
>>>
>>> On Wed, Dec 28, 2011 at 02:57, Gerhard Petracek
>>> <ge...@gmail.com>wrote:
>>>
>>>> +1
>>>>
>>>> regards,
>>>> gerhard
>>>>
>>>>
>>>>
>>>> 2011/12/28 Mark Struberg <st...@yahoo.de>
>>>>
>>>> > Hi!
>>>> >
>>>> > Whenever you spot the following sequence in a class:
>>>> >
>>>> > //X TODO some comment about the 'why'
>>>> >
>>>> >
>>>> > then it means that there is some unfinished stuff lying around...
>>>> >
>>>> > I'd say we now start adding stuff and mark all parts which
>> cause
>>>> > sentiments with such a comment and clean this mess up in a 2nd
>> step
>>>> later.
>>>> >
>>>> > LieGrue,
>>>> > strub
>>>> >
>>>> >
>>>>
>>>
>>>
>>>
>>> --
>>> Jason Porter
>>> http://lightguard-jp.blogspot.com
>>> http://twitter.com/lightguardjp
>>>
>>> Software Engineer
>>> Open Source Advocate
>>> Author of Seam Catch - Next Generation Java Exception Handling
>>>
>>> PGP key id: 926CCFF5
>>> PGP key available at: keyserver.net, pgp.mit.edu
>>
>>
>>
>> --
>> Matthias Wessendorf
>>
>> blog: http://matthiaswessendorf.wordpress.com/
>> sessions: http://www.slideshare.net/mwessendorf
>> twitter: http://twitter.com/mwessendorf
>>
--
Matthias Wessendorf
blog: http://matthiaswessendorf.wordpress.com/
sessions: http://www.slideshare.net/mwessendorf
twitter: http://twitter.com/mwessendorf
Re: //X TODO
Posted by Gerhard Petracek <ge...@gmail.com>.
i agree with mark.
(currently most todos are related to open discussions and we need to review
them in any case.)
regards,
gerhard
2012/1/4 Mark Struberg <st...@yahoo.de>
> Actually I like them - IF they are not used excessively.
>
>
> Often this is a review marker for myself or others...
> Of course they should not remain in the code for longer than necessary.
>
> I usually let maven generate a taglist which I work through regularly.
>
> LieGrue,
> strub
>
>
> ----- Original Message -----
> > From: Matthias Wessendorf <ma...@apache.org>
> > To: deltaspike-dev@incubator.apache.org
> > Cc:
> > Sent: Wednesday, January 4, 2012 4:09 PM
> > Subject: Re: //X TODO
> >
> > FWIW
> >
> > http://twitter.com/#!/abailly/status/154506876594290689
> >
> > -M
> >
> > On Tue, Jan 3, 2012 at 10:02 PM, Jason Porter <li...@gmail.com>
> > wrote:
> >> +1
> >>
> >> On Wed, Dec 28, 2011 at 02:57, Gerhard Petracek
> >> <ge...@gmail.com>wrote:
> >>
> >>> +1
> >>>
> >>> regards,
> >>> gerhard
> >>>
> >>>
> >>>
> >>> 2011/12/28 Mark Struberg <st...@yahoo.de>
> >>>
> >>> > Hi!
> >>> >
> >>> > Whenever you spot the following sequence in a class:
> >>> >
> >>> > //X TODO some comment about the 'why'
> >>> >
> >>> >
> >>> > then it means that there is some unfinished stuff lying around...
> >>> >
> >>> > I'd say we now start adding stuff and mark all parts which
> > cause
> >>> > sentiments with such a comment and clean this mess up in a 2nd
> > step
> >>> later.
> >>> >
> >>> > LieGrue,
> >>> > strub
> >>> >
> >>> >
> >>>
> >>
> >>
> >>
> >> --
> >> Jason Porter
> >> http://lightguard-jp.blogspot.com
> >> http://twitter.com/lightguardjp
> >>
> >> Software Engineer
> >> Open Source Advocate
> >> Author of Seam Catch - Next Generation Java Exception Handling
> >>
> >> PGP key id: 926CCFF5
> >> PGP key available at: keyserver.net, pgp.mit.edu
> >
> >
> >
> > --
> > Matthias Wessendorf
> >
> > blog: http://matthiaswessendorf.wordpress.com/
> > sessions: http://www.slideshare.net/mwessendorf
> > twitter: http://twitter.com/mwessendorf
> >
>
Re: //X TODO
Posted by Mark Struberg <st...@yahoo.de>.
Actually I like them - IF they are not used excessively.
Often this is a review marker for myself or others...
Of course they should not remain in the code for longer than necessary.
I usually let maven generate a taglist which I work through regularly.
LieGrue,
strub
----- Original Message -----
> From: Matthias Wessendorf <ma...@apache.org>
> To: deltaspike-dev@incubator.apache.org
> Cc:
> Sent: Wednesday, January 4, 2012 4:09 PM
> Subject: Re: //X TODO
>
> FWIW
>
> http://twitter.com/#!/abailly/status/154506876594290689
>
> -M
>
> On Tue, Jan 3, 2012 at 10:02 PM, Jason Porter <li...@gmail.com>
> wrote:
>> +1
>>
>> On Wed, Dec 28, 2011 at 02:57, Gerhard Petracek
>> <ge...@gmail.com>wrote:
>>
>>> +1
>>>
>>> regards,
>>> gerhard
>>>
>>>
>>>
>>> 2011/12/28 Mark Struberg <st...@yahoo.de>
>>>
>>> > Hi!
>>> >
>>> > Whenever you spot the following sequence in a class:
>>> >
>>> > //X TODO some comment about the 'why'
>>> >
>>> >
>>> > then it means that there is some unfinished stuff lying around...
>>> >
>>> > I'd say we now start adding stuff and mark all parts which
> cause
>>> > sentiments with such a comment and clean this mess up in a 2nd
> step
>>> later.
>>> >
>>> > LieGrue,
>>> > strub
>>> >
>>> >
>>>
>>
>>
>>
>> --
>> Jason Porter
>> http://lightguard-jp.blogspot.com
>> http://twitter.com/lightguardjp
>>
>> Software Engineer
>> Open Source Advocate
>> Author of Seam Catch - Next Generation Java Exception Handling
>>
>> PGP key id: 926CCFF5
>> PGP key available at: keyserver.net, pgp.mit.edu
>
>
>
> --
> Matthias Wessendorf
>
> blog: http://matthiaswessendorf.wordpress.com/
> sessions: http://www.slideshare.net/mwessendorf
> twitter: http://twitter.com/mwessendorf
>
Re: //X TODO
Posted by Matthias Wessendorf <ma...@apache.org>.
FWIW
http://twitter.com/#!/abailly/status/154506876594290689
-M
On Tue, Jan 3, 2012 at 10:02 PM, Jason Porter <li...@gmail.com> wrote:
> +1
>
> On Wed, Dec 28, 2011 at 02:57, Gerhard Petracek
> <ge...@gmail.com>wrote:
>
>> +1
>>
>> regards,
>> gerhard
>>
>>
>>
>> 2011/12/28 Mark Struberg <st...@yahoo.de>
>>
>> > Hi!
>> >
>> > Whenever you spot the following sequence in a class:
>> >
>> > //X TODO some comment about the 'why'
>> >
>> >
>> > then it means that there is some unfinished stuff lying around...
>> >
>> > I'd say we now start adding stuff and mark all parts which cause
>> > sentiments with such a comment and clean this mess up in a 2nd step
>> later.
>> >
>> > LieGrue,
>> > strub
>> >
>> >
>>
>
>
>
> --
> Jason Porter
> http://lightguard-jp.blogspot.com
> http://twitter.com/lightguardjp
>
> Software Engineer
> Open Source Advocate
> Author of Seam Catch - Next Generation Java Exception Handling
>
> PGP key id: 926CCFF5
> PGP key available at: keyserver.net, pgp.mit.edu
--
Matthias Wessendorf
blog: http://matthiaswessendorf.wordpress.com/
sessions: http://www.slideshare.net/mwessendorf
twitter: http://twitter.com/mwessendorf
Re: //X TODO
Posted by Jason Porter <li...@gmail.com>.
+1
On Wed, Dec 28, 2011 at 02:57, Gerhard Petracek
<ge...@gmail.com>wrote:
> +1
>
> regards,
> gerhard
>
>
>
> 2011/12/28 Mark Struberg <st...@yahoo.de>
>
> > Hi!
> >
> > Whenever you spot the following sequence in a class:
> >
> > //X TODO some comment about the 'why'
> >
> >
> > then it means that there is some unfinished stuff lying around...
> >
> > I'd say we now start adding stuff and mark all parts which cause
> > sentiments with such a comment and clean this mess up in a 2nd step
> later.
> >
> > LieGrue,
> > strub
> >
> >
>
--
Jason Porter
http://lightguard-jp.blogspot.com
http://twitter.com/lightguardjp
Software Engineer
Open Source Advocate
Author of Seam Catch - Next Generation Java Exception Handling
PGP key id: 926CCFF5
PGP key available at: keyserver.net, pgp.mit.edu
Re: //X TODO
Posted by Gerhard Petracek <ge...@gmail.com>.
+1
regards,
gerhard
2011/12/28 Mark Struberg <st...@yahoo.de>
> Hi!
>
> Whenever you spot the following sequence in a class:
>
> //X TODO some comment about the 'why'
>
>
> then it means that there is some unfinished stuff lying around...
>
> I'd say we now start adding stuff and mark all parts which cause
> sentiments with such a comment and clean this mess up in a 2nd step later.
>
> LieGrue,
> strub
>
>