Marc Weustink schreef:
Luis Rodrigues wrote:
Could someone please take a look ate this issue and commit my patch
since I really need that to be working.

http://www.freepascal.org/mantis/view.php?id=9491


Typical borland solution. Personally I think that grouping on group number is better. (maybe we can add that someday) Anyway patch looks ok, however I would group the or statements together. I find mixing or and and statements unreadable.

(A or B or C and D) isn't clear ((A or B or C) and D) is.


((A or B or C) and D) is more clear, but it is different too.

(A or B or (C and D)) is more clear equivalent of the first expression.

A: true
B: false
C: false
D: false

(A or B or C and D): true.
((A or B or C) and D): false.

I think ((A or B or C) and D) is what the bug fix should be, and so this patch is wrong.

Vincent

_________________________________________________________________
    To unsubscribe: mail [EMAIL PROTECTED] with
               "unsubscribe" as the Subject
  archives at http://www.lazarus.freepascal.org/mailarchives

Reply via email to