I must be tired trying to work on the RGB sliders because I don’t seem to be thinking straight. I’m beginning to think that with only one background thread it may be very difficult to get quick response even with the upgrade to the code using unsafe even with my little test images (forget it for sure working with large ones).

I’m spawning a background thread (don’t mention the overhead :( ) everytime the slider changes which begings the method to alter the red value of each pixel rather than “reset” the single thread which didn’t seem any better.

I have to get some actual times down to see if this is even better than just using one single background thread and cancelling the operation everytime the slider changes.

Anyway here’s some of the code so far of my sort of weak attempt to get fast feedback:

private void trackBar_RED_ValueChanged(object sender, EventArgs e)
{
  BackgroundWorker colorChangerWorker = new BackgroundWorker();
  colorChangerWorker.DoWork += new 
DoWorkEventHandler(colorChangerBackGroundWorker_DoWorkRed);
  colorChangerWorker.RunWorkerAsync
(new ColorChangingObject(trackBar_RED.Value, trackBar_RED.Maximum, colorChangerWorker,
 originalImage.Width, originalImage.Height, originalImage));
}
private void colorChangerBackGroundWorker_DoWorkRed(object sender, DoWorkEventArgs e)
{
  ColorChangingObject paramObject = (ColorChangingObject)e.Argument;
  double sliderValue = (double)paramObject.SliderValue;
  double maxValue = (double)paramObject.MaxSliderValue;
  double addRedPercent = sliderValue / maxValue;
  FastBitMapProcessor processImage = new FastBitMapProcessor(paramObject.Image);
  processImage.LockImage();
  for (int i = 0; i < paramObject.Width; i++)
  {
   for (int j = 0; j < paramObject.Height; j++)
    {
     Color currPixelColor = processImage.GetPixel(i, j);
     double red = (addRedPercent * (255 - currPixelColor.R)) + currPixelColor.R;
     processImage.SetPixel(i, j, Color.FromArgb((int)red, currPixelColor.G, currPixelColor.B));
    }
  }
  processImage.UnlockImage();
  changedImage = (Bitmap)processImage.Image.Clone();
  if (pictureBox2.InvokeRequired)
  {
   var UpdateImage = new   UpdatePictureBoxImage(SetPictureBox2Image);
   Invoke(UpdateImage, changedImage);
  }
  processImage.Dispose();
  paramObject.backgroundThread.Dispose();
}

Post any help if you have some.

Side Stuff

I guess I’ve made a lot of posts which are really really software oriented versus rather than some other stuff I’ve been doing lately (squash, ice hockey) I just found this project to be really interesting and on my mind.

Part of what is so interesting is noticing how a project envisoined initially so small can just explode if not checked. I seem to be working just as hard to fight thoughts to add the application than I do actually working on it. I guess that is what the benefit of programming something for an assigment versus than for you is. You tend to not go beyond the scope of the assignment, whereas for yourself you let your thoughts run free.

I feel this is something it is good to get a grip on. It seems to be a big problem some people lack in the software industry especially evidently from my current and previous jobs. People never want to freeze the software and label it feature complete; but rather seem to happy tacking on feature after feature.

---