Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.
Comment: Dynamic Task List: Task removed from list 'Review List'

...

Tasklist
Review List
Review List
||Completed||Priority||Locked||CreatedDate||CompletedDate||Assignee||Name||
|T|M|F|1270153016807|1270327838669|rcs_mgr|The title can also be Ensure classes containing publicly accessible mutable static fields are thread-safe|
|T|M|F|1270215734760|1270327843870|rcs_mgr|"Classes that use {{static}} fields that are both publicly accessible and mutable must always protect accesses to their fields, *regardless of their documentation.*" => A class that is internally providing synchronization would not document that it is not thread-safe. So I don't understand why you added the words in bold.|
|T|M|F|1270215935982|1270327856033|rcs_mgr|"The class relies on clients to externally synchronize the object and *this class documents* its lack of thread-safety" => "this class" occurs twice in the sentence. Should be "and documents its lack of thread safety".|
|T|M|F|1270325608947|1270327858018|rcs_mgr|"any assurances that the class is safe-thread."  => safe-thread should be thread-safe!|

...