Fatal error on calling a non-existing bundle

Patch applies cleanly and solves the problem mentioned.

A much clearer approach would be:

if ($bundle = icon_bundle_load($match[1])) {
  icon_process_attached($bundle);
}

re-rolled with suggestion.

Small, change, you're repeating a line of code unnecessarily here:

           if (!isset($icons[$match[0]])) {
               $bundle = icon_bundle_load($match[1]);
-              icon_process_attached($bundle);
+              if ($bundle = icon_bundle_load($match[1])) {
+                icon_process_attached($bundle);
+              }
             }

Should be:

           if (!isset($icons[$match[0]])) {
-              $bundle = icon_bundle_load($match[1]);
-              icon_process_attached($bundle);
+              if ($bundle = icon_bundle_load($match[1])) {
+                icon_process_attached($bundle);
+              }
             }

The original $bundle = icon_bundle_load($match[1]); can come out, since that's in the if statement as well.

You can always just re-roll the patch.

Looks good to me!

...adding the error message to the issue summary (and part of it in the issue title) so that people can find this issue easier when searching by error text.

#6 fixes this. Thanx @kyletaylored!!!

...missed a space there. Sorry for the noise.

Adding it to the issue summary is enough (it will be picked up in search). Changing the title back for legibility and preventing unnecessarily long issue table rows.